kate/BadFish
1
0
Fork 0
mirror of https://github.com/sockspls/badfish synced 2025-04-29 16:23:09 +00:00
Code Issues Releases Wiki Activity

Use for_each() in Endgames d'tor

Browse source 
And fix some comments while there.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
This commit is contained in:
Marco Costalba 2011-12-29 00:13:49 +01:00
parent 0759d8f430
commit 4a8c1b2470
3 changed files with 10 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/endgame.cpp
View file

@ -17,8 +17,8 @@
along with this program. If not, see <http://www.gnu.org/licenses/>. along with this program. If not, see <http://www.gnu.org/licenses/>.
*/ */
#include <cassert>
#include <algorithm> #include <algorithm>
#include <cassert>
#include "bitcount.h" #include "bitcount.h"
#include "endgame.h" #include "endgame.h"
@ -90,6 +90,9 @@ namespace {
return Position(fen, false, 0).material_key(); return Position(fen, false, 0).material_key();
} }
template<typename M>
void delete_endgame(const typename M::value_type& p) { delete p.second; }
} // namespace } // namespace
@ -119,11 +122,8 @@ Endgames::Endgames() {
Endgames::~Endgames() { Endgames::~Endgames() {
for (M1::const_iterator it = m1.begin(); it != m1.end(); ++it) for_each(m1.begin(), m1.end(), delete_endgame<M1>);
delete it->second; for_each(m2.begin(), m2.end(), delete_endgame<M2>);
for (M2::const_iterator it = m2.begin(); it != m2.end(); ++it)
delete it->second;
} }
template<EndgameType E> template<EndgameType E>

5
src/endgame.h
View file

@ -93,9 +93,9 @@ private:
/// Endgames class stores in two std::map the pointers to endgame evaluation /// Endgames class stores in two std::map the pointers to endgame evaluation
/// and scaling base objects. Then we use polymorphism to invoke the actual /// and scaling base objects. Then we use polymorphism to invoke the actual
/// endgame function calling its apply() method that is virtual. /// endgame function calling its operator() method that is virtual.
struct Endgames { class Endgames {
template<typename T> template<typename T>
struct Map { typedef std::map<Key, EndgameBase<T>*> type; }; struct Map { typedef std::map<Key, EndgameBase<T>*> type; };
@ -103,6 +103,7 @@ struct Endgames {
typedef Map<Value>::type M1; typedef Map<Value>::type M1;
typedef Map<ScaleFactor>::type M2; typedef Map<ScaleFactor>::type M2;
public:
Endgames(); Endgames();
~Endgames(); ~Endgames();

2
src/ucioption.cpp
View file

@ -38,7 +38,7 @@ bool CaseInsensitiveLess::operator() (const string& s1, const string& s2) const
/// OptionsMap c'tor initializes the UCI options to their hard coded default /// OptionsMap c'tor initializes the UCI options to their hard coded default
/// values and initializes the default value of "Threads" and "Minimum Split Depth" /// values and initializes the default value of "Threads" and "Min Split Depth"
/// parameters according to the number of CPU cores detected. /// parameters according to the number of CPU cores detected.
OptionsMap::OptionsMap() { OptionsMap::OptionsMap() {