widelands-dev team mailing list archive
Mailing list archive
[Merge] lp:~widelands-dev/widelands/bug-1800571-flag-add-ware into lp:widelands
Notabilis has proposed merging lp:~widelands-dev/widelands/bug-1800571-flag-add-ware into lp:widelands.
Undoing "optimization" that is part of previous commit in r8903.
Widelands Developers (widelands-dev)
Bug #1800571 in widelands: "crash with r8903"
For more details, see:
Commit r8903 moved some code within carrier.cc to avoid a duplicated call of a method in the Flag class. Unfortunately I overlooked that the call is no simple getter but modifies the internal state of the Flag, so it has to happen at the old code position.
Now doing the call to the method potentially twice, but it doesn't really matters performance-wise anyway.
For reference the old, buggy commit:
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1800571-flag-add-ware into lp:widelands.