widelands-dev team mailing list archive

[Hans Joachim Desserud <launchpad@xxxxxxxxxxxx>]

Hans Joachim Desserud has proposed merging lp:~hjd/widelands/remove-unused-variables into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~hjd/widelands/remove-unused-variables/+merge/50468

Removed a few unused variables which were causing warnings at compilation-time.
-- 
https://code.launchpad.net/~hjd/widelands/remove-unused-variables/+merge/50468
Your team Widelands Developers is requested to review the proposed merge of lp:~hjd/widelands/remove-unused-variables into lp:widelands.

=== modified file 'src/ai/defaultai.cc'
--- src/ai/defaultai.cc	2011-01-04 15:22:35 +0000
+++ src/ai/defaultai.cc	2011-02-19 20:44:03 +0000
@@ -1790,8 +1790,6 @@
 	container_iterate_const(std::set<Building_Index>, enhancements, x) {
 		// Only enhance buildings that are allowed (scenario mode)
 		if (player->is_building_type_allowed(*x.current)) {
-			Building_Descr const & bld = *tribe->get_building_descr(*x.current);
-			BuildingObserver & en_bo = get_building_observer(bld.name().c_str());
 
 			// Check if mine needs an enhancement to mine more resources
 			uint8_t const until =

=== modified file 'src/scripting/lua_game.cc'
--- src/scripting/lua_game.cc	2010-11-21 11:44:22 +0000
+++ src/scripting/lua_game.cc	2011-02-19 20:44:03 +0000
@@ -570,7 +570,6 @@
 */
 int L_Player::add_objective(lua_State * L) {
 	Game & game = get_game(L);
-	Player & p = get(L, game);
 
 	Map * map = game.get_map();
 	Manager<Objective> & mom = map->mom();