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

Revision history for this message
Notabilis (notabilis27) wrote :

I haven't tested it, but the code looks okay.

One could consider displaying an in-game message instead/additionally to the console output. The console is probably overlooked by players, but a change in the building definitions could lead to broken scenario save games. Personally I don't think that this is important, though, if such a case happens loading will probably break in other steps as well.

review: Approve (code)

« Back to merge proposal