widelands-dev team mailing list archive

[TiborB <tiborb95@xxxxxxxxx>]

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

Requested reviews:
  Widelands Developers (widelands-dev)

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

I feel a need to have explicit mention in log that this is emergency save - I think players sometimes confuse crash during emergency save with a crash during normal save. It should be obvious that the main problem is not gamesave.
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/emergency_log into lp:widelands.

=== modified file 'src/wlapplication.cc'
--- src/wlapplication.cc	2016-12-06 09:22:28 +0000
+++ src/wlapplication.cc	2016-12-09 20:07:46 +0000
@@ -1360,6 +1360,7 @@
 * Try to save the game instance if possible
  */
 void WLApplication::emergency_save(Widelands::Game& game) {
+	log("FATAL ERROR - game crashed. Attempting emergency save.\n");
 	if (game.is_loaded()) {
 		try {
 			SaveHandler& save_handler = game.save_handler();