[wesnoth-commits] [wesnoth/wesnoth] a46ddf: Game Events: don't write disabled events to config
GitHub
noreply at github.com
Tue Mar 6 06:38:07 UTC 2018
Branch: refs/heads/master
Home: https://github.com/wesnoth/wesnoth
Commit: a46ddf9bcd991b13c1c6ca19985b84b76a660ce8
https://github.com/wesnoth/wesnoth/commit/a46ddf9bcd991b13c1c6ca19985b84b76a660ce8
Author: Charles Dang <exodia339 at gmail.com>
Date: 2018-03-06 (Tue, 06 Mar 2018)
Changed paths:
M src/game_events/manager.cpp
Log Message:
-----------
Game Events: don't write disabled events to config
This should help mitigate an odd issue some people have reported with 1.13.11. After reloading a
save *after* executing a first_time_only=yes event, they can execute the event again. Looking at
savefiles for one of the reports, the event was included in the savefile even though it had already
executed. Most likely, when the player reloaded, the event was re-added to the gamestate and they
could execute it again.
The questions is then why the event had never been removed in handler cleanup. I'll have to look
into that further, but this at least ensures we don't get dead events written to config (that
includes saved games).
More information about the Commits
mailing list