widelands-dev team mailing list archive

[cghislai <charlyghislain@xxxxxxxxx>]

Ah, you merged already
I am pretty sure that it IS equal, but anyway in the last commit I put a when condition, as its more readable.
The best imho would be to put the 3 lines separatelty and only and complex conditions if it is really needed.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug988870/+merge/174639
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug988870 into lp:widelands.