widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #07528
Re: [Merge] lp:~widelands-dev/widelands/bug-1302593-result-screen into lp:widelands
I have added a question in the diff comments.
Diff comments:
>
> === modified file 'src/logic/playersmanager.cc'
> --- src/logic/playersmanager.cc 2016-02-16 10:27:23 +0000
> +++ src/logic/playersmanager.cc 2016-05-02 11:22:37 +0000
> @@ -120,5 +120,21 @@
> }
> }
>
> +void PlayersManager::set_player_end_status(const PlayerEndStatus& status)
> +{
> + bool found = false;
> + for (auto it = players_end_status_.begin(); it != players_end_status_.end(); ++it) {
> + PlayerEndStatus pes = *it;
> + if (pes.player == status.player) {
> + pes = status;
If this line is redundant, how do I change the status for an existing player status while loading a savegame?
> + found = true;
> + break;
> + }
> + }
> + if (!found) {
> + players_end_status_.push_back(status);
> + }
> +}
> +
>
> } // namespace Widelands
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1302593-result-screen/+merge/293521
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1302593-result-screen.
References