From 4310aaf5cc059bcf301e4e709866f336a3de3fcc Mon Sep 17 00:00:00 2001 From: Johann Dreo Date: Mon, 1 Nov 2010 22:09:40 +0100 Subject: [PATCH] =?UTF-8?q?replace=20TODO=20comments=20by=20@=C5=A7odo=20d?= =?UTF-8?q?oxygen=20command?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- eo/src/eoFlOrBinOp.h | 2 +- eo/src/eoFlOrQuadOp.h | 2 +- eo/src/eoSTLFunctor.h | 2 +- eo/src/eoVariableLengthCrossover.h | 2 +- eo/src/es/eoCMABreed.h | 2 +- eo/src/es/eoCMAInit.h | 2 +- eo/src/gp/parse_tree.h | 4 ++-- eo/src/utils/eoFileSnapshot.h | 2 +- eo/src/utils/eoOStreamMonitor.cpp | 3 ++- eo/src/utils/eoParser.cpp | 4 ++-- 10 files changed, 13 insertions(+), 12 deletions(-) diff --git a/eo/src/eoFlOrBinOp.h b/eo/src/eoFlOrBinOp.h index 124ca4fd..2dda394d 100644 --- a/eo/src/eoFlOrBinOp.h +++ b/eo/src/eoFlOrBinOp.h @@ -109,7 +109,7 @@ public : } bool changed = false; - for ( unsigned i = 0; i < k; i++ ) // TODO: check that we don't do twice the same + for ( unsigned i = 0; i < k; i++ ) //! @todo check that we don't do twice the same { unsigned where = eo::rng.random(_eo1.size()); bool changedHere = op( _eo1[where], _eo2[where] ); diff --git a/eo/src/eoFlOrQuadOp.h b/eo/src/eoFlOrQuadOp.h index 1ad873c4..0e4b9a46 100644 --- a/eo/src/eoFlOrQuadOp.h +++ b/eo/src/eoFlOrQuadOp.h @@ -101,7 +101,7 @@ public : } bool changed = false; - for ( unsigned i = 0; i < k; i++ ) // TODO: check that we don't do twice the same + for ( unsigned i = 0; i < k; i++ ) //! @todo check that we don't do twice the same { unsigned where = eo::rng.random(_eo1.size()); bool changedHere = op( _eo1[where], _eo2[where] ); diff --git a/eo/src/eoSTLFunctor.h b/eo/src/eoSTLFunctor.h index 74c20603..ca8b9233 100644 --- a/eo/src/eoSTLFunctor.h +++ b/eo/src/eoSTLFunctor.h @@ -113,6 +113,6 @@ class eoSTLBF : public std::binary_function eoBF& f; }; -// TODO: put automated wrappers here... +//! @todo: put automated wrappers here... #endif diff --git a/eo/src/eoVariableLengthCrossover.h b/eo/src/eoVariableLengthCrossover.h index ff86c60f..9d604ea8 100644 --- a/eo/src/eoVariableLengthCrossover.h +++ b/eo/src/eoVariableLengthCrossover.h @@ -251,7 +251,7 @@ public : } while ( ( (tmp1.size()Max) || (tmp2.size()>Max) ) && (index<10000) ); - // FIXME bad hardcoded limit, should use an algorithm that guarantee a correct size in a finite number of tries + //! @todo FIXME bad hardcoded limit, should use an algorithm that guarantee a correct size in a finite number of tries if (index >= 10000) { eo::log << eo::warnings << "Warning: impossible to generate individual of the right size in 10000 trials" << std::endl; diff --git a/eo/src/es/eoCMABreed.h b/eo/src/es/eoCMABreed.h index 1c998c42..d55ab261 100644 --- a/eo/src/es/eoCMABreed.h +++ b/eo/src/es/eoCMABreed.h @@ -31,7 +31,7 @@ #include -/// TODO, handle bounds +/// @todo handle bounds template class eoCMABreed : public eoBreed< eoVector > { diff --git a/eo/src/es/eoCMAInit.h b/eo/src/es/eoCMAInit.h index b605cd99..bce9e5ba 100644 --- a/eo/src/es/eoCMAInit.h +++ b/eo/src/es/eoCMAInit.h @@ -32,7 +32,7 @@ #include #include -/// TODO, handle bounds +/// @todo handle bounds template class eoCMAInit : public eoInit< eoVector > { diff --git a/eo/src/gp/parse_tree.h b/eo/src/gp/parse_tree.h index ba4a3861..0e85b576 100644 --- a/eo/src/gp/parse_tree.h +++ b/eo/src/gp/parse_tree.h @@ -181,9 +181,9 @@ template class parse_tree class subtree { -/* +/** a bit nasty way to use a pool allocator (which would otherwise use slooow new and delete) - TODO: use the std::allocator interface + @todo use the std::allocator interface */ #if (defined(__GNUC__) || defined(_MSC_VER)) && !(defined(_MT) || defined(MACOSX) || defined(__APPLE__)) // not multithreaded (or MACOSX - J. Eggermont) diff --git a/eo/src/utils/eoFileSnapshot.h b/eo/src/utils/eoFileSnapshot.h index 7392c70a..c9e58c74 100644 --- a/eo/src/utils/eoFileSnapshot.h +++ b/eo/src/utils/eoFileSnapshot.h @@ -43,7 +43,7 @@ Assumes that the parameters that are passed to the monitor A dir is created and one file per snapshot is created there - so you can later generate a movie! -TODO: The counter is handled internally, but this should be changed +@todo The counter is handled internally, but this should be changed so that you can pass e.g. an evalcounter (minor) I failed to templatize everything so that it can handle eoParam > diff --git a/eo/src/utils/eoOStreamMonitor.cpp b/eo/src/utils/eoOStreamMonitor.cpp index 073793d8..f47096d0 100644 --- a/eo/src/utils/eoOStreamMonitor.cpp +++ b/eo/src/utils/eoOStreamMonitor.cpp @@ -37,7 +37,8 @@ eoMonitor& eoOStreamMonitor::operator()(void) } // if firstime // ok, now the real saving. write out - /* TODO old verbose formatting, do we still need it? + //! @todo old verbose formatting, do we still need it? + /* for (iterator it = vec.begin (); it != vec.end (); ++it) { // name: value out << (*it)->longName () << ": " << (*it)->getValue () << std::endl; diff --git a/eo/src/utils/eoParser.cpp b/eo/src/utils/eoParser.cpp index baefc48f..f6c81f14 100644 --- a/eo/src/utils/eoParser.cpp +++ b/eo/src/utils/eoParser.cpp @@ -164,7 +164,7 @@ pair eoParser::getValue(eoParam& _param) const result.first = true; return result; } - // else (TODO: check environment, just long names) + //! @todo check environment, just long names return result; } @@ -210,7 +210,7 @@ void eoParser::readFrom(istream& is) string value; if (equalLocation == str.end()) - { // TODO: it should be the next string + { //! @todo it should be the next string value = ""; } else