diff --git a/lemon/bfs.h b/lemon/bfs.h --- a/lemon/bfs.h +++ b/lemon/bfs.h @@ -53,7 +53,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient to initialize static PredMap *createPredMap(const Digraph &g) { return new PredMap(g); @@ -63,7 +62,6 @@ ///The type of the map that indicates which nodes are processed. ///It must meet the \ref concepts::WriteMap "WriteMap" concept. - ///By default it is a NullMap. typedef NullMap ProcessedMap; ///Instantiates a \ref ProcessedMap. @@ -195,8 +193,7 @@ int _queue_head,_queue_tail,_queue_next_dist; int _curr_dist; - ///Creates the maps if necessary. - ///\todo Better memory allocation (instead of new). + //Creates the maps if necessary. void create_maps() { if(!_pred) { @@ -847,7 +844,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient to initialize #ifdef DOXYGEN static PredMap *createPredMap(const Digraph &g) #else @@ -1332,8 +1328,7 @@ std::vector _list; int _list_front, _list_back; - ///Creates the maps if necessary. - ///\todo Better memory allocation (instead of new). + //Creates the maps if necessary. void create_maps() { if(!_reached) { local_reached = true; diff --git a/lemon/bits/base_extender.h b/lemon/bits/base_extender.h --- a/lemon/bits/base_extender.h +++ b/lemon/bits/base_extender.h @@ -105,9 +105,6 @@ /// Returns whether the given directed arc has the same orientation /// as the corresponding edge. - /// - /// \todo reference to the corresponding point of the undirected digraph - /// concept. "What does the direction of an edge mean?" static bool direction(const Arc &a) { return a.forward; } using Parent::first; diff --git a/lemon/bits/vector_map.h b/lemon/bits/vector_map.h --- a/lemon/bits/vector_map.h +++ b/lemon/bits/vector_map.h @@ -42,10 +42,9 @@ /// automatically updates the map when a key is added to or erased from /// the map. This map type uses the std::vector to store the values. /// - /// \tparam _Notifier The AlterationNotifier that will notify this map. + /// \tparam _Graph The graph this map is attached to. /// \tparam _Item The item type of the graph items. /// \tparam _Value The value type of the map. - /// \todo Fix the doc: there is _Graph parameter instead of _Notifier. template class VectorMap : public ItemSetTraits<_Graph, _Item>::ItemNotifier::ObserverBase { diff --git a/lemon/concept_check.h b/lemon/concept_check.h --- a/lemon/concept_check.h +++ b/lemon/concept_check.h @@ -36,8 +36,6 @@ ///\file ///\brief Basic utilities for concept checking. /// -///\todo Are we still using BOOST concept checking utility? -///Is the BOOST copyright notice necessary? #ifndef LEMON_CONCEPT_CHECK_H #define LEMON_CONCEPT_CHECK_H diff --git a/lemon/concepts/path.h b/lemon/concepts/path.h --- a/lemon/concepts/path.h +++ b/lemon/concepts/path.h @@ -20,7 +20,6 @@ ///\file ///\brief Classes for representing paths in digraphs. /// -///\todo Iterators have obsolete style #ifndef LEMON_CONCEPT_PATH_H #define LEMON_CONCEPT_PATH_H diff --git a/lemon/dfs.h b/lemon/dfs.h --- a/lemon/dfs.h +++ b/lemon/dfs.h @@ -54,7 +54,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient to initialize static PredMap *createPredMap(const Digraph &g) { return new PredMap(g); @@ -64,7 +63,6 @@ ///The type of the map that indicates which nodes are processed. ///It must meet the \ref concepts::WriteMap "WriteMap" concept. - ///By default it is a NullMap. typedef NullMap ProcessedMap; ///Instantiates a \ref ProcessedMap. @@ -195,8 +193,7 @@ std::vector _stack; int _stack_head; - ///Creates the maps if necessary. - ///\todo Better memory allocation (instead of new). + //Creates the maps if necessary. void create_maps() { if(!_pred) { @@ -782,7 +779,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient to initialize #ifdef DOXYGEN static PredMap *createPredMap(const Digraph &g) #else @@ -1279,8 +1275,7 @@ std::vector _stack; int _stack_head; - ///Creates the maps if necessary. - ///\todo Better memory allocation (instead of new). + //Creates the maps if necessary. void create_maps() { if(!_reached) { local_reached = true; diff --git a/lemon/dijkstra.h b/lemon/dijkstra.h --- a/lemon/dijkstra.h +++ b/lemon/dijkstra.h @@ -143,7 +143,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient for the initialization static PredMap *createPredMap(const Digraph &g) { return new PredMap(g); @@ -154,8 +153,6 @@ ///The type of the map that indicates which nodes are processed. ///It must meet the \ref concepts::WriteMap "WriteMap" concept. ///By default it is a NullMap. - ///\todo If it is set to a real map, - ///Dijkstra::processed() should read this. typedef NullMap ProcessedMap; ///Instantiates a \ref ProcessedMap. @@ -296,8 +293,7 @@ //Indicates if _heap is locally allocated (true) or not. bool local_heap; - ///Creates the maps if necessary. - ///\todo Better memory allocation (instead of new). + //Creates the maps if necessary. void create_maps() { if(!_pred) { @@ -957,7 +953,6 @@ ///This function instantiates a \ref HeapCrossRef. /// \param g is the digraph, to which we would like to define the /// HeapCrossRef. - /// \todo The digraph alone may be insufficient for the initialization static HeapCrossRef *createHeapCrossRef(const Digraph &g) { return new HeapCrossRef(g); @@ -993,7 +988,6 @@ ///This function instantiates a \ref PredMap. ///\param g is the digraph, to which we would like to define the ///\ref PredMap. - ///\todo The digraph alone may be insufficient to initialize #ifdef DOXYGEN static PredMap *createPredMap(const Digraph &g) #else @@ -1008,9 +1002,6 @@ ///The type of the map that indicates which nodes are processed. ///It must meet the \ref concepts::WriteMap "WriteMap" concept. ///By default it is a NullMap. - ///\todo If it is set to a real map, - ///Dijkstra::processed() should read this. - ///\todo named parameter to set this type, function to read and write. typedef NullMap ProcessedMap; ///Instantiates a \ref ProcessedMap. @@ -1054,7 +1045,6 @@ /// as well as the \ref Dijkstra class. /// The \ref DijkstraWizardBase is a class to be the default traits of the /// \ref DijkstraWizard class. - /// \todo More named parameters are required... template class DijkstraWizardBase : public DijkstraWizardDefaultTraits { diff --git a/lemon/error.h b/lemon/error.h --- a/lemon/error.h +++ b/lemon/error.h @@ -102,8 +102,6 @@ protected: ///\e - ///\todo The good solution is boost::shared_ptr... - /// mutable std::auto_ptr buf; ///\e diff --git a/lemon/graph_to_eps.h b/lemon/graph_to_eps.h --- a/lemon/graph_to_eps.h +++ b/lemon/graph_to_eps.h @@ -666,7 +666,6 @@ ///this function calls the algorithm itself, i.e. in this case ///it draws the graph. void run() { - //\todo better 'epsilon' would be nice here. const double EPSILON=1e-9; if(dontPrint) return; @@ -707,7 +706,6 @@ double max_w=0; for(ArcIt e(g);e!=INVALID;++e) max_w=std::max(double(_arcWidths[e]),max_w); - //\todo better 'epsilon' would be nice here. if(max_w>EPSILON) { _arcWidthScale/=max_w; } @@ -717,7 +715,6 @@ double max_s=0; for(NodeIt n(g);n!=INVALID;++n) max_s=std::max(double(_nodeSizes[n]),max_s); - //\todo better 'epsilon' would be nice here. if(max_s>EPSILON) { _nodeScale/=max_s; } @@ -873,7 +870,6 @@ << -bb.left() << ' ' << -bb.bottom() << " translate\n"; } else { - //\todo Verify centering double sc= std::min((A4HEIGHT-2*A4BORDER)/bb.width(), (A4WIDTH-2*A4BORDER)/bb.height()); os << ((A4WIDTH -2*A4BORDER)-sc*bb.height())/2 + A4BORDER << ' ' @@ -906,7 +902,6 @@ dim2::Point dvec(mycoords[g.target(*i)]-mycoords[g.source(*i)]); double l=std::sqrt(dvec.normSquare()); - //\todo better 'epsilon' would be nice here. dim2::Point d(dvec/std::max(l,EPSILON)); dim2::Point m; // m=dim2::Point(mycoords[g.target(*i)]+ diff --git a/lemon/list_graph.h b/lemon/list_graph.h --- a/lemon/list_graph.h +++ b/lemon/list_graph.h @@ -501,10 +501,8 @@ ///valid. However InArcIts and OutArcIts may ///be invalidated. /// - ///\warning This functionality cannot be used together with the + ///\warning This functionality cannot be used in conjunction with the ///Snapshot feature. - /// - ///\todo It could be implemented in a bit faster way. Node split(Node n, bool connect = true) { Node b = addNode(); for(OutArcIt e(*this,n);e!=INVALID;) { diff --git a/lemon/maps.h b/lemon/maps.h --- a/lemon/maps.h +++ b/lemon/maps.h @@ -484,8 +484,6 @@ /// function. /// /// \sa CombineMap - /// - /// \todo Check the requirements. template class ComposeMap : public MapBase { const M1 &_m1; @@ -540,8 +538,6 @@ /// function. /// /// \sa ComposeMap - /// - /// \todo Check the requirements. template class CombineMap : public MapBase { diff --git a/lemon/random.h b/lemon/random.h --- a/lemon/random.h +++ b/lemon/random.h @@ -821,7 +821,6 @@ /// Standard Gauss distribution. /// \note The Cartesian form of the Box-Muller /// transformation is used to generate a random normal distribution. - /// \todo Consider using the "ziggurat" method instead. double gauss() { double V1,V2,S; diff --git a/lemon/smart_graph.h b/lemon/smart_graph.h --- a/lemon/smart_graph.h +++ b/lemon/smart_graph.h @@ -300,7 +300,6 @@ ///may be invalidated. ///\warning This functionality cannot be used together with the Snapshot ///feature. - ///\todo It could be implemented in a bit faster way. Node split(Node n, bool connect = true) { Node b = addNode(); diff --git a/lemon/time_measure.h b/lemon/time_measure.h --- a/lemon/time_measure.h +++ b/lemon/time_measure.h @@ -292,7 +292,6 @@ ///\note If you want to measure the running time of the execution of a certain ///function, consider the usage of \ref TimeReport instead. /// - ///\todo This shouldn't be Unix (Linux) specific. ///\sa TimeReport class Timer { @@ -487,7 +486,6 @@ /// ///\sa Timer ///\sa NoTimeReport - ///\todo There is no test case for this class TimeReport : public Timer { std::string _title; diff --git a/lemon/tolerance.h b/lemon/tolerance.h --- a/lemon/tolerance.h +++ b/lemon/tolerance.h @@ -24,8 +24,6 @@ ///\brief A basic tool to handle the anomalies of calculation with ///floating point numbers. /// -///\todo It should be in a module like "Basic tools" - namespace lemon {