Code review comment for lp:~widelands-dev/widelands/economy-target-profiles

Revision history for this message
Toni Förster (stonerl) wrote :

One major problem. Pressing the save button pauses the game without actually showing, that the game is paused. That might tolerable in single player, but in multiplayer it pauses the game for all participants. This should not happen.

Also get this warning during compiling:

src/wui/economy_options_window.h:40:2: warning: '~EconomyOptionsWindow' overrides a destructor but is not marked 'override'

review: Needs Fixing

« Back to merge proposal