Code review comment for lp:~widelands-dev/widelands/bug543113

Revision history for this message
Jens Beyer (qcumber-some) wrote :

Unchanged. Somehow I lost track of this in a heavy-work phase ;-)

Problem is still the "Needs fixing" topic.
I could merge the two if statements to one, no problem.

But I can not decide on the design of the win condition. All I did was, implement what I found in the win condition.

Someone has to decide if the win condition should be reworked (and if it should, then this statistics should be updated too). Maybe it is a topic for a different bug entry? Then we could merge this one (it still merges without errors.. ;-) )

« Back to merge proposal