Code review comment for lp://staging/~widelands-dev/widelands/bug-1191295-no-seafaring-builder

Revision history for this message
GunChleoc (gunchleoc) wrote :

Increasing the version number of the portdock is a good idea. Rather than the ugly hack, you could pass the version number of the portdock to the load function.

You could also check if you can still separate wares from workers in a way that allows you to keep the save format, as long as we don't end up with overcomplicated code.

Games from Build 19 should still load after this change too.

« Back to merge proposal