widelands

Code review comment for lp:~hjd/widelands/remove-unused-variables

  1. remove-unused-variables
  2. Merge into trunk
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

FWIW, here's the warnings:
/home/user/widelands/src/ai/defaultai.cc: In member function ‘bool DefaultAI::check_mines(int32_t)’:
/home/user/widelands/src/ai/defaultai.cc:1794: warning: unused variable ‘en_bo’
/home/user/widelands/src/scripting/lua_game.cc: In member function ‘int LuaGame::L_Player::add_objective(lua_State*)’:
/home/user/widelands/src/scripting/lua_game.cc:573: warning: unused variable ‘p’

The warning concerned bld appeared once I had removed en_bo, since it was the only place where it was used.

« Back to merge proposal