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

Revision history for this message
GunChleoc (gunchleoc) wrote :

You can now get rid of the second_phase variable entirely. Just check for !anything_selected at the end of the loop, then you can also get rid of

  if (anything_selected) {
   return;
  }

Confirmed that the layouting is fixed. The ware statistics doesn't fit yet at 800x600 resolution, can you give it 1 less row for that?

Also added 1 comment.

« Back to merge proposal