Changes in lemon/bits/graph_extender.h [78:c46b3453455f:57:c1acf0018c0a] in lemon-1.2

File:

• lemon/bits/graph_extender.h (modified) (37 diffs)

lemon/bits/graph_extender.h

@@ -21 +21 @@
 
 #include <lemon/bits/invalid.h>
-#include <lemon/bits/utility.h>
 
 #include <lemon/bits/map_extender.h>
@@ -65 +64 @@
     }
 
-    Node oppositeNode(const Node &node, const Arc &arc) const {
-      if (node == Parent::source(arc))
-        return Parent::target(arc);
-      else if(node == Parent::target(arc))
-        return Parent::source(arc);
+    Node oppositeNode(const Node &n, const Arc &e) const {
+      if (n == Parent::source(e))
+        return Parent::target(e);
+      else if(n==Parent::target(e))
+        return Parent::source(e);
       else
         return INVALID;
@@ -96 +95 @@
 
     class NodeIt : public Node { 
-      const Digraph* _digraph;
+      const Digraph* digraph;
     public:
 
@@ -103 +102 @@
       NodeIt(Invalid i) : Node(i) { }
 
-      explicit NodeIt(const Digraph& digraph) : _digraph(&digraph) {
-        _digraph->first(static_cast<Node&>(*this));
-      }
-
-      NodeIt(const Digraph& digraph, const Node& node) 
-        : Node(node), _digraph(&digraph) {}
+      explicit NodeIt(const Digraph& _digraph) : digraph(&_digraph) {
+        _digraph.first(static_cast<Node&>(*this));
+      }
+
+      NodeIt(const Digraph& _digraph, const Node& node) 
+        : Node(node), digraph(&_digraph) {}
 
       NodeIt& operator++() { 
-        _digraph->next(*this);
+        digraph->next(*this);
         return *this; 
       }
@@ -119 +118 @@
 
     class ArcIt : public Arc { 
-      const Digraph* _digraph;
+      const Digraph* digraph;
     public:
 
@@ -126 +125 @@
       ArcIt(Invalid i) : Arc(i) { }
 
-      explicit ArcIt(const Digraph& digraph) : _digraph(&digraph) {
-        _digraph->first(static_cast<Arc&>(*this));
-      }
-
-      ArcIt(const Digraph& digraph, const Arc& arc) : 
-        Arc(arc), _digraph(&digraph) { }
+      explicit ArcIt(const Digraph& _digraph) : digraph(&_digraph) {
+        _digraph.first(static_cast<Arc&>(*this));
+      }
+
+      ArcIt(const Digraph& _digraph, const Arc& e) : 
+        Arc(e), digraph(&_digraph) { }
 
       ArcIt& operator++() { 
-        _digraph->next(*this);
+        digraph->next(*this);
         return *this; 
       }
@@ -142 +141 @@
 
     class OutArcIt : public Arc { 
-      const Digraph* _digraph;
+      const Digraph* digraph;
     public:
 
@@ -149 +148 @@
       OutArcIt(Invalid i) : Arc(i) { }
 
-      OutArcIt(const Digraph& digraph, const Node& node) 
-        : _digraph(&digraph) {
-        _digraph->firstOut(*this, node);
-      }
-
-      OutArcIt(const Digraph& digraph, const Arc& arc) 
-        : Arc(arc), _digraph(&digraph) {}
+      OutArcIt(const Digraph& _digraph, const Node& node) 
+        : digraph(&_digraph) {
+        _digraph.firstOut(*this, node);
+      }
+
+      OutArcIt(const Digraph& _digraph, const Arc& arc) 
+        : Arc(arc), digraph(&_digraph) {}
 
       OutArcIt& operator++() { 
-        _digraph->nextOut(*this);
+        digraph->nextOut(*this);
         return *this; 
       }
@@ -166 +165 @@
 
     class InArcIt : public Arc { 
-      const Digraph* _digraph;
+      const Digraph* digraph;
     public:
 
@@ -173 +172 @@
       InArcIt(Invalid i) : Arc(i) { }
 
-      InArcIt(const Digraph& digraph, const Node& node) 
-        : _digraph(&digraph) {
-        _digraph->firstIn(*this, node);
-      }
-
-      InArcIt(const Digraph& digraph, const Arc& arc) : 
-        Arc(arc), _digraph(&digraph) {}
+      InArcIt(const Digraph& _digraph, const Node& node) 
+        : digraph(&_digraph) {
+        _digraph.firstIn(*this, node);
+      }
+
+      InArcIt(const Digraph& _digraph, const Arc& arc) : 
+        Arc(arc), digraph(&_digraph) {}
 
       InArcIt& operator++() { 
-        _digraph->nextIn(*this);
+        digraph->nextIn(*this);
         return *this; 
       }
@@ -191 +190 @@
     ///
     /// Returns the base node (i.e. the source in this case) of the iterator
-    Node baseNode(const OutArcIt &arc) const {
-      return Parent::source(arc);
+    Node baseNode(const OutArcIt &e) const {
+      return Parent::source(e);
     }
     /// \brief Running node of the iterator
@@ -198 +197 @@
     /// Returns the running node (i.e. the target in this case) of the
     /// iterator
-    Node runningNode(const OutArcIt &arc) const {
-      return Parent::target(arc);
+    Node runningNode(const OutArcIt &e) const {
+      return Parent::target(e);
     }
 
@@ -205 +204 @@
     ///
     /// Returns the base node (i.e. the target in this case) of the iterator
-    Node baseNode(const InArcIt &arc) const {
-      return Parent::target(arc);
+    Node baseNode(const InArcIt &e) const {
+      return Parent::target(e);
     }
     /// \brief Running node of the iterator
@@ -212 +211 @@
     /// Returns the running node (i.e. the source in this case) of the
     /// iterator
-    Node runningNode(const InArcIt &arc) const {
-      return Parent::source(arc);
+    Node runningNode(const InArcIt &e) const {
+      return Parent::source(e);
     }
 
@@ -325 +324 @@
   };
 
-  /// \ingroup _graphbits
+  /// \ingroup graphbits
   ///
   /// \brief Extender for the Graphs
@@ -333 +332 @@
     
     typedef Base Parent;
-    typedef GraphExtender Graph;
-
-    typedef True UndirectedTag;
+    typedef GraphExtender Digraph;
 
     typedef typename Parent::Node Node;
@@ -376 +373 @@
     }
 
-    Arc oppositeArc(const Arc &arc) const {
-      return Parent::direct(arc, !Parent::direction(arc));
+    Arc oppositeArc(const Arc &e) const {
+      return Parent::direct(e, !Parent::direction(e));
     }
 
     using Parent::direct;
-    Arc direct(const Edge &edge, const Node &node) const {
-      return Parent::direct(edge, Parent::source(edge) == node);
+    Arc direct(const Edge &ue, const Node &s) const {
+      return Parent::direct(ue, Parent::source(ue) == s);
     }
 
@@ -415 +412 @@
 
     class NodeIt : public Node { 
-      const Graph* _graph;
+      const Digraph* digraph;
     public:
 
@@ -422 +419 @@
       NodeIt(Invalid i) : Node(i) { }
 
-      explicit NodeIt(const Graph& graph) : _graph(&graph) {
-        _graph->first(static_cast<Node&>(*this));
-      }
-
-      NodeIt(const Graph& graph, const Node& node) 
-        : Node(node), _graph(&graph) {}
+      explicit NodeIt(const Digraph& _digraph) : digraph(&_digraph) {
+        _digraph.first(static_cast<Node&>(*this));
+      }
+
+      NodeIt(const Digraph& _digraph, const Node& node) 
+        : Node(node), digraph(&_digraph) {}
 
       NodeIt& operator++() { 
-        _graph->next(*this);
+        digraph->next(*this);
         return *this; 
       }
@@ -438 +435 @@
 
     class ArcIt : public Arc { 
-      const Graph* _graph;
+      const Digraph* digraph;
     public:
 
@@ -445 +442 @@
       ArcIt(Invalid i) : Arc(i) { }
 
-      explicit ArcIt(const Graph& graph) : _graph(&graph) {
-        _graph->first(static_cast<Arc&>(*this));
-      }
-
-      ArcIt(const Graph& graph, const Arc& arc) : 
-        Arc(arc), _graph(&graph) { }
+      explicit ArcIt(const Digraph& _digraph) : digraph(&_digraph) {
+        _digraph.first(static_cast<Arc&>(*this));
+      }
+
+      ArcIt(const Digraph& _digraph, const Arc& e) : 
+        Arc(e), digraph(&_digraph) { }
 
       ArcIt& operator++() { 
-        _graph->next(*this);
+        digraph->next(*this);
         return *this; 
       }
@@ -461 +458 @@
 
     class OutArcIt : public Arc { 
-      const Graph* _graph;
+      const Digraph* digraph;
     public:
 
@@ -468 +465 @@
       OutArcIt(Invalid i) : Arc(i) { }
 
-      OutArcIt(const Graph& graph, const Node& node) 
-        : _graph(&graph) {
-        _graph->firstOut(*this, node);
-      }
-
-      OutArcIt(const Graph& graph, const Arc& arc) 
-        : Arc(arc), _graph(&graph) {}
+      OutArcIt(const Digraph& _digraph, const Node& node) 
+        : digraph(&_digraph) {
+        _digraph.firstOut(*this, node);
+      }
+
+      OutArcIt(const Digraph& _digraph, const Arc& arc) 
+        : Arc(arc), digraph(&_digraph) {}
 
       OutArcIt& operator++() { 
-        _graph->nextOut(*this);
+        digraph->nextOut(*this);
         return *this; 
       }
@@ -485 +482 @@
 
     class InArcIt : public Arc { 
-      const Graph* _graph;
+      const Digraph* digraph;
     public:
 
@@ -492 +489 @@
       InArcIt(Invalid i) : Arc(i) { }
 
-      InArcIt(const Graph& graph, const Node& node) 
-        : _graph(&graph) {
-        _graph->firstIn(*this, node);
-      }
-
-      InArcIt(const Graph& graph, const Arc& arc) : 
-        Arc(arc), _graph(&graph) {}
+      InArcIt(const Digraph& _digraph, const Node& node) 
+        : digraph(&_digraph) {
+        _digraph.firstIn(*this, node);
+      }
+
+      InArcIt(const Digraph& _digraph, const Arc& arc) : 
+        Arc(arc), digraph(&_digraph) {}
 
       InArcIt& operator++() { 
-        _graph->nextIn(*this);
+        digraph->nextIn(*this);
         return *this; 
       }
@@ -509 +506 @@
 
     class EdgeIt : public Parent::Edge { 
-      const Graph* _graph;
+      const Digraph* digraph;
     public:
 
@@ -516 +513 @@
       EdgeIt(Invalid i) : Edge(i) { }
 
-      explicit EdgeIt(const Graph& graph) : _graph(&graph) {
-        _graph->first(static_cast<Edge&>(*this));
-      }
-
-      EdgeIt(const Graph& graph, const Edge& edge) : 
-        Edge(edge), _graph(&graph) { }
+      explicit EdgeIt(const Digraph& _digraph) : digraph(&_digraph) {
+        _digraph.first(static_cast<Edge&>(*this));
+      }
+
+      EdgeIt(const Digraph& _digraph, const Edge& e) : 
+        Edge(e), digraph(&_digraph) { }
 
       EdgeIt& operator++() { 
-        _graph->next(*this);
-        return *this; 
-      }
-
-    };
-
-    class IncEdgeIt : public Parent::Edge {
+        digraph->next(*this);
+        return *this; 
+      }
+
+    };
+
+    class IncArcIt : public Parent::Edge {
       friend class GraphExtender;
-      const Graph* _graph;
-      bool _direction;
-    public:
-
-      IncEdgeIt() { }
-
-      IncEdgeIt(Invalid i) : Edge(i), _direction(false) { }
-
-      IncEdgeIt(const Graph& graph, const Node &node) : _graph(&graph) {
-        _graph->firstInc(*this, _direction, node);
-      }
-
-      IncEdgeIt(const Graph& graph, const Edge &edge, const Node &node)
-        : _graph(&graph), Edge(edge) {
-        _direction = (_graph->source(edge) == node);
-      }
-
-      IncEdgeIt& operator++() {
-        _graph->nextInc(*this, _direction);
+      const Digraph* digraph;
+      bool direction;
+    public:
+
+      IncArcIt() { }
+
+      IncArcIt(Invalid i) : Edge(i), direction(false) { }
+
+      IncArcIt(const Digraph& _digraph, const Node &n) : digraph(&_digraph) {
+        _digraph.firstInc(*this, direction, n);
+      }
+
+      IncArcIt(const Digraph& _digraph, const Edge &ue, const Node &n)
+        : digraph(&_digraph), Edge(ue) {
+        direction = (_digraph.source(ue) == n);
+      }
+
+      IncArcIt& operator++() {
+        digraph->nextInc(*this, direction);
         return *this; 
       }
@@ -558 +555 @@
     ///
     /// Returns the base node (ie. the source in this case) of the iterator
-    Node baseNode(const OutArcIt &arc) const {
-      return Parent::source(static_cast<const Arc&>(arc));
+    Node baseNode(const OutArcIt &e) const {
+      return Parent::source(static_cast<const Arc&>(e));
     }
     /// \brief Running node of the iterator
@@ -565 +562 @@
     /// Returns the running node (ie. the target in this case) of the
     /// iterator
-    Node runningNode(const OutArcIt &arc) const {
-      return Parent::target(static_cast<const Arc&>(arc));
+    Node runningNode(const OutArcIt &e) const {
+      return Parent::target(static_cast<const Arc&>(e));
     }
 
@@ -572 +569 @@
     ///
     /// Returns the base node (ie. the target in this case) of the iterator
-    Node baseNode(const InArcIt &arc) const {
-      return Parent::target(static_cast<const Arc&>(arc));
+    Node baseNode(const InArcIt &e) const {
+      return Parent::target(static_cast<const Arc&>(e));
     }
     /// \brief Running node of the iterator
@@ -579 +576 @@
     /// Returns the running node (ie. the source in this case) of the
     /// iterator
-    Node runningNode(const InArcIt &arc) const {
-      return Parent::source(static_cast<const Arc&>(arc));
+    Node runningNode(const InArcIt &e) const {
+      return Parent::source(static_cast<const Arc&>(e));
     }
 
@@ -586 +583 @@
     ///
     /// Returns the base node of the iterator
-    Node baseNode(const IncEdgeIt &edge) const {
-      return edge._direction ? source(edge) : target(edge);
+    Node baseNode(const IncArcIt &e) const {
+      return e.direction ? source(e) : target(e);
     }
     /// Running node of the iterator
     ///
     /// Returns the running node of the iterator
-    Node runningNode(const IncEdgeIt &edge) const {
-      return edge._direction ? target(edge) : source(edge);
+    Node runningNode(const IncArcIt &e) const {
+      return e.direction ? target(e) : source(e);
     }
 
@@ -600 +597 @@
     template <typename _Value>
     class NodeMap 
-      : public MapExtender<DefaultMap<Graph, Node, _Value> > {
-    public:
-      typedef GraphExtender Graph;
-      typedef MapExtender<DefaultMap<Graph, Node, _Value> > Parent;
-
-      NodeMap(const Graph& graph) 
-        : Parent(graph) {}
-      NodeMap(const Graph& graph, const _Value& value) 
-        : Parent(graph, value) {}
+      : public MapExtender<DefaultMap<Digraph, Node, _Value> > {
+    public:
+      typedef GraphExtender Digraph;
+      typedef MapExtender<DefaultMap<Digraph, Node, _Value> > Parent;
+
+      NodeMap(const Digraph& digraph) 
+        : Parent(digraph) {}
+      NodeMap(const Digraph& digraph, const _Value& value) 
+        : Parent(digraph, value) {}
 
       NodeMap& operator=(const NodeMap& cmap) {
@@ -624 +621 @@
     template <typename _Value>
     class ArcMap 
-      : public MapExtender<DefaultMap<Graph, Arc, _Value> > {
-    public:
-      typedef GraphExtender Graph;
-      typedef MapExtender<DefaultMap<Graph, Arc, _Value> > Parent;
-
-      ArcMap(const Graph& graph) 
-        : Parent(graph) {}
-      ArcMap(const Graph& graph, const _Value& value) 
-        : Parent(graph, value) {}
+      : public MapExtender<DefaultMap<Digraph, Arc, _Value> > {
+    public:
+      typedef GraphExtender Digraph;
+      typedef MapExtender<DefaultMap<Digraph, Arc, _Value> > Parent;
+
+      ArcMap(const Digraph& digraph) 
+        : Parent(digraph) {}
+      ArcMap(const Digraph& digraph, const _Value& value) 
+        : Parent(digraph, value) {}
 
       ArcMap& operator=(const ArcMap& cmap) {
@@ -648 +645 @@
     template <typename _Value>
     class EdgeMap 
-      : public MapExtender<DefaultMap<Graph, Edge, _Value> > {
-    public:
-      typedef GraphExtender Graph;
-      typedef MapExtender<DefaultMap<Graph, Edge, _Value> > Parent;
-
-      EdgeMap(const Graph& graph) 
-        : Parent(graph) {}
-
-      EdgeMap(const Graph& graph, const _Value& value) 
-        : Parent(graph, value) {}
+      : public MapExtender<DefaultMap<Digraph, Edge, _Value> > {
+    public:
+      typedef GraphExtender Digraph;
+      typedef MapExtender<DefaultMap<Digraph, Edge, _Value> > Parent;
+
+      EdgeMap(const Digraph& digraph) 
+        : Parent(digraph) {}
+
+      EdgeMap(const Digraph& digraph, const _Value& value) 
+        : Parent(digraph, value) {}
 
       EdgeMap& operator=(const EdgeMap& cmap) {
@@ -696 +693 @@
     }
 
-    template <typename Graph, typename NodeRefMap, typename EdgeRefMap>
-    void build(const Graph& graph, NodeRefMap& nodeRef, 
+    template <typename Digraph, typename NodeRefMap, typename EdgeRefMap>
+    void build(const Digraph& digraph, NodeRefMap& nodeRef, 
                EdgeRefMap& edgeRef) {
-      Parent::build(graph, nodeRef, edgeRef);
+      Parent::build(digraph, nodeRef, edgeRef);
       notifier(Node()).build();
       notifier(Edge()).build();
@@ -724 +721 @@
 
     void erase(const Edge& edge) {
-      std::vector<Arc> av;
-      av.push_back(Parent::direct(edge, true));
-      av.push_back(Parent::direct(edge, false));      
-      notifier(Arc()).erase(av);
+      std::vector<Arc> ev;
+      ev.push_back(Parent::direct(edge, true));
+      ev.push_back(Parent::direct(edge, false));      
+      notifier(Arc()).erase(ev);
       notifier(Edge()).erase(edge);
       Parent::erase(edge);