]> git.lyx.org Git - lyx.git/blobdiff - src/Graph.cpp
Seems boost also includes all std headers. Not including boost therefore produces...
[lyx.git] / src / Graph.cpp
index 7a185efd763b4807484a9384233e0e54fa870c6e..da7170b2cba91a050a4a11c139bf460e896742ec 100644 (file)
@@ -3,7 +3,8 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Dekel Tsur
+ * \author Dekel Tsur (original code)
+ * \author Richard Heck (re-implementation)
  *
  * Full author contact details are available in file CREDITS.
  */
@@ -13,6 +14,9 @@
 #include "Graph.h"
 #include "Format.h"
 
+#include "support/debug.h"
+#include "support/lassert.h"
+
 #include <algorithm>
 
 using namespace std;
@@ -41,6 +45,15 @@ bool Graph::bfs_init(int s, bool clear_visited)
 }
 
 
+void Graph::clearMarks()
+{
+       Arrows::iterator it = arrows_.begin();
+       Arrows::iterator const en = arrows_.end();
+       for (; it != en; ++it)
+               it->marked = false;
+}
+
+
 vector<int> const
        Graph::getReachableTo(int target, bool clear_visited)
 {
@@ -50,7 +63,7 @@ vector<int> const
 
        // Here's the logic, which is shared by the other routines.
        // Q_ holds a list of nodes we have been able to reach (in this
-       // case, reach backwards). It is initailized to the current node
+       // case, reach backwards). It is initialized to the current node
        // by bfs_init, and then we recurse, adding the nodes we can reach
        // from the current node as we go. That makes it a breadth-first
        // search.
@@ -60,12 +73,13 @@ vector<int> const
                if (current != target || formats.get(target).name() != "lyx")
                        result.push_back(current);
 
-               vector<int>::iterator it = vertices_[current].in_vertices.begin();
-               vector<int>::iterator end = vertices_[current].in_vertices.end();
+               vector<Arrow *>::iterator it = vertices_[current].in_arrows.begin();
+               vector<Arrow *>::iterator const end = vertices_[current].in_arrows.end();
                for (; it != end; ++it) {
-                       if (!vertices_[*it].visited) {
-                               vertices_[*it].visited = true;
-                               Q_.push(*it);
+                       const int cv = (*it)->from;
+                       if (!vertices_[cv].visited) {
+                               vertices_[cv].visited = true;
+                               Q_.push(cv);
                        }
                }
        }
@@ -95,12 +109,12 @@ vector<int> const
                                result.push_back(current);
                }
 
-               vector<OutEdge>::const_iterator cit =
+               vector<Arrow *>::const_iterator cit =
                        vertices_[current].out_arrows.begin();
-               vector<OutEdge>::const_iterator end =
+               vector<Arrow *>::const_iterator end =
                        vertices_[current].out_arrows.end();
                for (; cit != end; ++cit) {
-                       int const cv = cit->vertex;
+                       int const cv = (*cit)->to;
                        if (!vertices_[cv].visited) {
                                vertices_[cv].visited = true;
                                Q_.push(cv);
@@ -126,12 +140,12 @@ bool Graph::isReachable(int from, int to)
                if (current == to)
                        return true;
 
-               vector<OutEdge>::const_iterator cit =
+               vector<Arrow *>::const_iterator cit =
                        vertices_[current].out_arrows.begin();
-               vector<OutEdge>::const_iterator end =
+               vector<Arrow *>::const_iterator end =
                        vertices_[current].out_arrows.end();
                for (; cit != end; ++cit) {
-                       int const cv = cit->vertex;
+                       int const cv = (*cit)->to;
                        if (!vertices_[cv].visited) {
                                vertices_[cv].visited = true;
                                Q_.push(cv);
@@ -152,29 +166,28 @@ Graph::EdgePath const Graph::getPath(int from, int to)
        if (to < 0 || !bfs_init(from))
                return path;
 
-       // pair<vertex, edge>
-       vector<pair<int, int> > prev(vertices_.size());
-
+       // In effect, the way this works is that we construct a sub-graph
+       // by starting at "from" and following the arrows outward. Instead
+       // of actually constructing a sub-graph, though, we "mark" the
+       // arrows we traverse as we go. Once we hit "to", we abort the 
+       // marking process and then call getMarkedPath() to reconstruct
+       // the marked path.
        bool found = false;
+       clearMarks();
        while (!Q_.empty()) {
                int const current = Q_.front();
                Q_.pop();
 
-               vector<OutEdge>::const_iterator const beg =
+               vector<Arrow *>::const_iterator cit =
                        vertices_[current].out_arrows.begin();
-               vector<OutEdge>::const_iterator cit = beg;
-               vector<OutEdge>::const_iterator end =
+               vector<Arrow *>::const_iterator end =
                        vertices_[current].out_arrows.end();
                for (; cit != end; ++cit) {
-                       int const cv = cit->vertex;
+                       int const cv = (*cit)->to;
                        if (!vertices_[cv].visited) {
                                vertices_[cv].visited = true;
                                Q_.push(cv);
-                               // FIXME This will not do for finding multiple paths.
-                               // Perhaps we need a vector, or a set. We'll also want
-                               // to add this info, even if the node is visited, so
-                               // outside this conditional.
-                               prev[cv] = pair<int, int>(current, cit->edge);
+                               (*cit)->marked = true;
                        }
                        if (cv == to) {
                                found = true;
@@ -185,26 +198,78 @@ Graph::EdgePath const Graph::getPath(int from, int to)
        if (!found)
                return path;
 
-       while (to != from) {
-               path.push_back(prev[to].second);
-               to = prev[to].first;
-       }
-       reverse(path.begin(), path.end());
+       getMarkedPath(from, to, path);
        return path;
 }
 
 
+// We assume we have marked the graph, as in getPath(). We also
+// assume that we have done so in such a way as to guarantee a
+// marked path from "from" to "to".
+// We then start at "to" and find the arrow leading to it that
+// has been marked. We add that to the path we are constructing,
+// step back on that arrow, and continue the process (i.e., recurse).
+void Graph::getMarkedPath(int from, int to, EdgePath & path) {
+       if (from == to) {
+               reverse(path.begin(), path.end());
+               return;
+       }
+       // find marked in_arrow
+       vector<Arrow *>::const_iterator it = vertices_[to].in_arrows.begin();
+       vector<Arrow *>::const_iterator const en = vertices_[to].in_arrows.end();
+       for (; it != en; ++it)
+               if ((*it)->marked) 
+                       break;
+       if (it == en) {
+               // debug code to try to figure out what's up.
+               LYXERR0("Failed to find marked arrow.\n"
+                                               "From: " << from << ", To: " << to);
+               dumpGraph();
+               LASSERT(false, /* */);
+               return;
+       }
+       path.push_back((*it)->id);
+       getMarkedPath(from, (*it)->from, path);
+}
+
+       
 void Graph::init(int size)
 {
        vertices_ = vector<Vertex>(size);
+       arrows_.clear();
        numedges_ = 0;
 }
 
 
 void Graph::addEdge(int from, int to)
 {
-       vertices_[to].in_vertices.push_back(from);
-       vertices_[from].out_arrows.push_back(OutEdge(to, numedges_++));
+       arrows_.push_back(Arrow(from, to, numedges_));
+       numedges_++;
+       Arrow * ar = &(arrows_.back());
+       vertices_[to].in_arrows.push_back(ar);
+       vertices_[from].out_arrows.push_back(ar);
+}
+
+
+void Graph::dumpGraph() const
+{
+       vector<Vertex>::const_iterator it = vertices_.begin();
+       vector<Vertex>::const_iterator en = vertices_.end();
+       for (; it != en; ++it) {
+               LYXERR0("Next vertex...");
+               LYXERR0("In arrows...");
+               std::vector<Arrow *>::const_iterator iit = it->in_arrows.begin();
+               std::vector<Arrow *>::const_iterator ien = it->in_arrows.end();
+               for (; iit != ien; ++iit)
+                       LYXERR0("From " << (*iit)->from << " to " << (*iit)->to
+                                       << ". Marked: " << (*iit)->marked);
+               LYXERR0("Out arrows...");
+               iit = it->out_arrows.begin();
+               ien = it->out_arrows.end();
+               for (; iit != ien; ++iit)
+                       LYXERR0("From " << (*iit)->from << " to " << (*iit)->to
+                                               << ". Marked: " << (*iit)->marked);
+       }
 }