diff options
author | Oliver Galvin <odg@riseup.net> | 2020-06-17 22:57:08 +0100 |
---|---|---|
committer | John Fultz <jfultz@wolfram.com> | 2020-06-18 12:19:44 -0500 |
commit | 82ec129c63e8287017c9321a8a5a54e5d52a921d (patch) | |
tree | 3c406832f893e91f3e729807f4bfbd2e934526da | |
parent | b7413384aafca679584df1e5e18dd7e6282ff4f3 (diff) |
Fix a couple of compiler warnings: -Wsign-compare and -Wpedantic
-rw-r--r-- | endgame.cpp | 12 | ||||
-rw-r--r-- | game.h | 8 |
2 files changed, 10 insertions, 10 deletions
diff --git a/endgame.cpp b/endgame.cpp index b41261a..5ebd7c6 100644 --- a/endgame.cpp +++ b/endgame.cpp @@ -216,8 +216,8 @@ double Endgame::disappoint(EndgameMove &hope, double bestPessimistic) m_endgameGame.setCandidate(hope.move); m_endgameGame.commitCandidate(true); - int levelNumber = 1; - int playerNumber = 1; + size_t levelNumber = 1; + size_t playerNumber = 1; while (!m_endgameGame.currentPosition().gameOver()) { @@ -304,8 +304,8 @@ Move Endgame::solve(int /* nestedness */) double beforeSpread = m_endgameGame.currentPosition().spread(startPlayerId); - int levelNumber = 1; - int playerNumber = 1; + size_t levelNumber = 1; + size_t playerNumber = 1; while (!m_endgameGame.currentPosition().gameOver()) { @@ -408,8 +408,8 @@ void Endgame::reallyPlayOut(Move &chosenMove, int nestedness) Game playoutGame = m_originalGame; - int levelNumber = 1; - int playerNumber = 1; + size_t levelNumber = 1; + size_t playerNumber = 1; double beforeSpread = playoutGame.currentPosition().spread(startPlayerId); @@ -451,22 +451,22 @@ protected: inline const Player &GamePosition::currentPlayer() const { return *m_currentPlayer; -}; +} inline Player &GamePosition::currentPlayer() { return *m_currentPlayer; -}; +} inline const Player &GamePosition::playerOnTurn() const { return *m_playerOnTurn; -}; +} inline Player &GamePosition::playerOnTurn() { return *m_playerOnTurn; -}; +} inline const MoveList &GamePosition::moves() const { |