Merge lp:~widelands-dev/widelands/bug-1791605-on-failure-fail into lp:widelands

[GunChleoc]

Remove test logging output from production_program.cc

[kaputtnik (franku)]

The messages are gone. I think this change is uncritical, so:

@bunnybot merge

review: Approve

[bunnybot (widelandsofficial)]

Continuous integration builds have changed state:

Travis build 4117. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/440586787.
Appveyor build 3914. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1791605_on_failure_fail-3914.

=== modified file 'src/logic/map_objects/tribes/production_program.cc'
--- src/logic/map_objects/tribes/production_program.cc	2018-09-10 12:32:56 +0000
+++ src/logic/map_objects/tribes/production_program.cc	2018-10-12 10:31:38 +0000
@@ -583,7 +583,6 @@
 		while (!reached_end) {
 			skip(parameters);
 			match_force_skip(parameters, "on");
-			log("found \"on \": parameters = \"%s\"\n", parameters);
 
 			ProgramResult result_to_set_method_for;
 			if (match_force_skip(parameters, "failure")) {
@@ -616,9 +615,6 @@
 				                    "{\"fail\"|\"complete\"|\"skip\"|\"repeat\"}", parameters);
 			handling_methods_[result_to_set_method_for - 1] = handling_method;
 			reached_end = !*parameters;
-			log("read handling method for result %u: %u, parameters = \"%s\", "
-			    "reached_end = %u\n",
-			    result_to_set_method_for, handling_method, parameters, reached_end);
 		}
 	} catch (const WException& e) {
 		throw GameDataError("call: %s", e.what());