widelands-dev team mailing list archive

[Jukka Pakarinen <flegmaatikko@xxxxxxxxx>]

Jukka Pakarinen has proposed merging lp:~widelands-dev/widelands/enemysites_check_delay_used_before_initialized into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1734544 in widelands: "Member variable DefaultAI::enemysites_check_delay_ used before initialization"
  https://bugs.launchpad.net/widelands/+bug/1734544

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/enemysites_check_delay_used_before_initialized/+merge/334287

DefaultAI::enemysites_check_delay_ variable is not initialized before it is used. 
The fix in the branch fixes the bug 1734544.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/enemysites_check_delay_used_before_initialized into lp:widelands.

=== modified file 'src/ai/defaultai.cc'
--- src/ai/defaultai.cc	2017-11-20 07:54:19 +0000
+++ src/ai/defaultai.cc	2017-11-26 17:19:38 +0000
@@ -106,6 +106,7 @@
      ts_in_const_count_(0),
      ts_without_trainers_(0),
      inhibit_road_building_(0),
+     enemysites_check_delay_(0),
      resource_necessity_water_needed_(false),
      highest_nonmil_prio_(0),
      seafaring_economy(false),