# HG changeset patch
# User Alpar Juttner <alpar@cs.elte.hu>
# Date 1221195475 -3600
# Node ID 6a55577420eb39c20730420a5f0d4e281703b032
# Parent c691064dfd4f2e3d4e230a1fd9845a1d96de2a5d
Copy constr and op= of the default maps has been mad private (see #137)
diff --git a/lemon/bits/array_map.h b/lemon/bits/array_map.h
|
a
|
b
|
|
| 103 | 103 | } |
| 104 | 104 | } |
| 105 | 105 | |
| | 106 | private: |
| 106 | 107 | /// \brief Constructor to copy a map of the same map type. |
| 107 | 108 | /// |
| 108 | 109 | /// Constructor to copy a map of the same map type. |
| … |
… |
|
| 150 | 151 | return *this; |
| 151 | 152 | } |
| 152 | 153 | |
| | 154 | public: |
| 153 | 155 | /// \brief The destructor of the map. |
| 154 | 156 | /// |
| 155 | 157 | /// The destructor of the map. |
diff --git a/lemon/bits/graph_extender.h b/lemon/bits/graph_extender.h
|
a
|
b
|
|
| 227 | 227 | NodeMap(const Digraph& digraph, const _Value& value) |
| 228 | 228 | : Parent(digraph, value) {} |
| 229 | 229 | |
| | 230 | private: |
| 230 | 231 | NodeMap& operator=(const NodeMap& cmap) { |
| 231 | 232 | return operator=<NodeMap>(cmap); |
| 232 | 233 | } |
| … |
… |
|
| 251 | 252 | ArcMap(const Digraph& digraph, const _Value& value) |
| 252 | 253 | : Parent(digraph, value) {} |
| 253 | 254 | |
| | 255 | private: |
| 254 | 256 | ArcMap& operator=(const ArcMap& cmap) { |
| 255 | 257 | return operator=<ArcMap>(cmap); |
| 256 | 258 | } |
| … |
… |
|
| 608 | 610 | NodeMap(const Graph& graph, const _Value& value) |
| 609 | 611 | : Parent(graph, value) {} |
| 610 | 612 | |
| | 613 | private: |
| 611 | 614 | NodeMap& operator=(const NodeMap& cmap) { |
| 612 | 615 | return operator=<NodeMap>(cmap); |
| 613 | 616 | } |
| … |
… |
|
| 632 | 635 | ArcMap(const Graph& graph, const _Value& value) |
| 633 | 636 | : Parent(graph, value) {} |
| 634 | 637 | |
| | 638 | private: |
| 635 | 639 | ArcMap& operator=(const ArcMap& cmap) { |
| 636 | 640 | return operator=<ArcMap>(cmap); |
| 637 | 641 | } |
| … |
… |
|
| 657 | 661 | EdgeMap(const Graph& graph, const _Value& value) |
| 658 | 662 | : Parent(graph, value) {} |
| 659 | 663 | |
| | 664 | private: |
| 660 | 665 | EdgeMap& operator=(const EdgeMap& cmap) { |
| 661 | 666 | return operator=<EdgeMap>(cmap); |
| 662 | 667 | } |
diff --git a/lemon/bits/map_extender.h b/lemon/bits/map_extender.h
|
a
|
b
|
|
| 62 | 62 | MapExtender(const Graph& graph, const Value& value) |
| 63 | 63 | : Parent(graph, value) {} |
| 64 | 64 | |
| | 65 | private: |
| 65 | 66 | MapExtender& operator=(const MapExtender& cmap) { |
| 66 | 67 | return operator=<MapExtender>(cmap); |
| 67 | 68 | } |
| … |
… |
|
| 72 | 73 | return *this; |
| 73 | 74 | } |
| 74 | 75 | |
| | 76 | public: |
| 75 | 77 | class MapIt : public Item { |
| 76 | 78 | public: |
| 77 | 79 | |
| … |
… |
|
| 200 | 202 | SubMapExtender(const Graph& _graph, const Value& _value) |
| 201 | 203 | : Parent(_graph, _value), graph(_graph) {} |
| 202 | 204 | |
| | 205 | private: |
| 203 | 206 | SubMapExtender& operator=(const SubMapExtender& cmap) { |
| 204 | 207 | return operator=<MapExtender>(cmap); |
| 205 | 208 | } |
| … |
… |
|
| 214 | 217 | return *this; |
| 215 | 218 | } |
| 216 | 219 | |
| | 220 | public: |
| 217 | 221 | class MapIt : public Item { |
| 218 | 222 | public: |
| 219 | 223 | |
diff --git a/lemon/bits/vector_map.h b/lemon/bits/vector_map.h
|
a
|
b
|
|
| 100 | 100 | container.resize(Parent::notifier()->maxId() + 1, value); |
| 101 | 101 | } |
| 102 | 102 | |
| | 103 | private: |
| 103 | 104 | /// \brief Copy constructor |
| 104 | 105 | /// |
| 105 | 106 | /// Copy constructor. |
diff --git a/lemon/concept_check.h b/lemon/concept_check.h
|
a
|
b
|
|
| 67 | 67 | inline void checkConcept() { |
| 68 | 68 | #if !defined(NDEBUG) |
| 69 | 69 | typedef typename Concept::template Constraints<Type> ConceptCheck; |
| 70 | | void (ConceptCheck::*x)() = & ConceptCheck::constraints; |
| 71 | | ignore_unused_variable_warning(x); |
| | 70 | // void (ConceptCheck::*x)() = & ConceptCheck::constraints; |
| | 71 | // ignore_unused_variable_warning(x); |
| 72 | 72 | #endif |
| 73 | 73 | } |
| 74 | 74 | |
diff --git a/lemon/concepts/digraph.h b/lemon/concepts/digraph.h
|
a
|
b
|
|
| 434 | 434 | ///\e |
| 435 | 435 | NodeMap(const Digraph&, T) { } |
| 436 | 436 | |
| | 437 | private: |
| 437 | 438 | ///Copy constructor |
| 438 | 439 | NodeMap(const NodeMap& nm) : ReadWriteMap< Node, T >(nm) { } |
| 439 | 440 | ///Assignment operator |
| … |
… |
|
| 456 | 457 | ArcMap(const Digraph&) { } |
| 457 | 458 | ///\e |
| 458 | 459 | ArcMap(const Digraph&, T) { } |
| | 460 | private: |
| 459 | 461 | ///Copy constructor |
| 460 | 462 | ArcMap(const ArcMap& em) : ReadWriteMap<Arc,T>(em) { } |
| 461 | 463 | ///Assignment operator |
diff --git a/lemon/concepts/graph.h b/lemon/concepts/graph.h
|
a
|
b
|
|
| 512 | 512 | ///\e |
| 513 | 513 | NodeMap(const Graph&, T) { } |
| 514 | 514 | |
| | 515 | private: |
| 515 | 516 | ///Copy constructor |
| 516 | 517 | NodeMap(const NodeMap& nm) : ReadWriteMap< Node, T >(nm) { } |
| 517 | 518 | ///Assignment operator |
| … |
… |
|
| 535 | 536 | ArcMap(const Graph&) { } |
| 536 | 537 | ///\e |
| 537 | 538 | ArcMap(const Graph&, T) { } |
| | 539 | private: |
| 538 | 540 | ///Copy constructor |
| 539 | 541 | ArcMap(const ArcMap& em) : ReadWriteMap<Arc,T>(em) { } |
| 540 | 542 | ///Assignment operator |
| … |
… |
|
| 558 | 560 | EdgeMap(const Graph&) { } |
| 559 | 561 | ///\e |
| 560 | 562 | EdgeMap(const Graph&, T) { } |
| | 563 | private: |
| 561 | 564 | ///Copy constructor |
| 562 | 565 | EdgeMap(const EdgeMap& em) : ReadWriteMap<Edge,T>(em) {} |
| 563 | 566 | ///Assignment operator |
diff --git a/lemon/concepts/graph_components.h b/lemon/concepts/graph_components.h
|
a
|
b
|
|
| 1030 | 1030 | // Constructor with a graph and a default value parameter |
| 1031 | 1031 | _Map a2(g,t); |
| 1032 | 1032 | // Copy constructor. |
| 1033 | | _Map b(c); |
| | 1033 | // _Map b(c); |
| 1034 | 1034 | |
| 1035 | 1035 | ReadMap<Key, Value> cmap; |
| 1036 | | b = cmap; |
| | 1036 | // b = cmap; |
| 1037 | 1037 | |
| 1038 | 1038 | ignore_unused_variable_warning(a2); |
| 1039 | | ignore_unused_variable_warning(b); |
| | 1039 | // ignore_unused_variable_warning(b); |
| 1040 | 1040 | } |
| 1041 | 1041 | |
| 1042 | 1042 | const _Map &c; |
| … |
… |
|
| 1082 | 1082 | NodeMap(const MappableDigraphComponent& digraph, const _Value& value) |
| 1083 | 1083 | : Parent(digraph, value) {} |
| 1084 | 1084 | |
| | 1085 | private: |
| 1085 | 1086 | /// \brief Copy constructor. |
| 1086 | 1087 | /// |
| 1087 | 1088 | /// Copy Constructor. |
| … |
… |
|
| 1119 | 1120 | ArcMap(const MappableDigraphComponent& digraph, const _Value& value) |
| 1120 | 1121 | : Parent(digraph, value) {} |
| 1121 | 1122 | |
| | 1123 | private: |
| 1122 | 1124 | /// \brief Copy constructor. |
| 1123 | 1125 | /// |
| 1124 | 1126 | /// Copy Constructor. |
| … |
… |
|
| 1215 | 1217 | EdgeMap(const MappableGraphComponent& graph, const _Value& value) |
| 1216 | 1218 | : Parent(graph, value) {} |
| 1217 | 1219 | |
| | 1220 | private: |
| 1218 | 1221 | /// \brief Copy constructor. |
| 1219 | 1222 | /// |
| 1220 | 1223 | /// Copy Constructor. |
diff --git a/test/graph_test.h b/test/graph_test.h
|
a
|
b
|
|
| 212 | 212 | } |
| 213 | 213 | check(s == 0, "Wrong sum."); |
| 214 | 214 | |
| 215 | | map = constMap<Node>(12); |
| 216 | | for (NodeIt it(G); it != INVALID; ++it) { |
| 217 | | check(map[it] == 12, "Wrong operator[]."); |
| 218 | | } |
| | 215 | // map = constMap<Node>(12); |
| | 216 | // for (NodeIt it(G); it != INVALID; ++it) { |
| | 217 | // check(map[it] == 12, "Wrong operator[]."); |
| | 218 | // } |
| 219 | 219 | } |
| 220 | 220 | |
| 221 | 221 | template <typename Graph> |
| … |
… |
|
| 242 | 242 | } |
| 243 | 243 | check(s == 0, "Wrong sum."); |
| 244 | 244 | |
| 245 | | map = constMap<Arc>(12); |
| 246 | | for (ArcIt it(G); it != INVALID; ++it) { |
| 247 | | check(map[it] == 12, "Wrong operator[]."); |
| 248 | | } |
| | 245 | // map = constMap<Arc>(12); |
| | 246 | // for (ArcIt it(G); it != INVALID; ++it) { |
| | 247 | // check(map[it] == 12, "Wrong operator[]."); |
| | 248 | // } |
| 249 | 249 | } |
| 250 | 250 | |
| 251 | 251 | template <typename Graph> |
| … |
… |
|
| 272 | 272 | } |
| 273 | 273 | check(s == 0, "Wrong sum."); |
| 274 | 274 | |
| 275 | | map = constMap<Edge>(12); |
| 276 | | for (EdgeIt it(G); it != INVALID; ++it) { |
| 277 | | check(map[it] == 12, "Wrong operator[]."); |
| 278 | | } |
| | 275 | // map = constMap<Edge>(12); |
| | 276 | // for (EdgeIt it(G); it != INVALID; ++it) { |
| | 277 | // check(map[it] == 12, "Wrong operator[]."); |
| | 278 | // } |
| 279 | 279 | } |
| 280 | 280 | |
| 281 | 281 | |