Show a message when the game is autosaving

Bug #738895 reported by Hans Joachim Desserud on 2011-03-20
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Wishlist
cghislai

Bug Description

(Split out from bug 738643)

Currently when the game is autosaving, especially on large maps, it may take some time. While this happens the player does not know what is going on and may think the game froze for a short while. When I discovered it I checked the terminal output to see that it was in fact saving the game, which regular players should not need to do.

Originally written by LAZA:
"there should also comes up a message or dialog window if the game is auto-saved.
i sometimes wonder, why the game slows down and/or is apruptly stopped for the automatic save.
if it is a bigger map and 2, 3 enemies the saving last for some seconds and the player wonders what is happening."
Originally written by me:
I fully agree with providing some feedback during autosaving. I actually noticed this yesterday and planned to report it, but you beat me to it. :)
"I thought about using the chat system or something similar to print some feedback, like "Autosaving..." when it starts, and "Saved" when it is finished. That way, the user will know what the game is doing, while today the game only seems to freeze when the autosave happens, especially on large maps."

Related branches

Changed in widelands:
status: New → Confirmed
importance: Undecided → Wishlist
tags: added: savegame ui
Joachim Breitner (nomeata) wrote :

Some random bits from looking at the code: The object that handles the display of chat messages (Interactive_GameBase.m_chatOverlay) is currently not created if we do not have a multiplayer session. If that is done, one needs to identify the cleanest way for logic code (src/save_handler.cc) to get hold of the ChatProvider that was passed to the chatOverlay, then you are almost done. But I’m going to bed now :-)

Changed in widelands:
assignee: nobody → cghislai (charlyghislain)
status: Confirmed → In Progress
cghislai (charlyghislain) wrote :

Please not that currently it will display a message only for autosaves, not manual ones.

Changed in widelands:
status: In Progress → Fix Committed
Changed in widelands:
milestone: none → build18-rc1
SirVer (sirver) wrote :

Released in build-18 rc1.

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers