widelands-dev team mailing list archive

[TiborB <tiborb95@xxxxxxxxx>]

TiborB has proposed merging lp:~widelands-dev/widelands/ai_return_added into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

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

This return was missed by our automated compilation setup (I have got emails about compilation failure). Bit strange because at my 2 boxes it never complained about this. 

Is there any flag for gcc to force this check?
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_return_added into lp:widelands.

=== modified file 'src/ai/defaultai.cc'
--- src/ai/defaultai.cc	2015-10-19 18:40:48 +0000
+++ src/ai/defaultai.cc	2015-10-24 19:57:28 +0000
@@ -3895,6 +3895,7 @@
 		// impossible but still
 		assert(false);
 	}
+	return BuildingNecessity::kNotNeeded;
 }
 
 // Now we can prohibit some militarysites, based on size, the goal is not to