widelands-dev team mailing list archive

[Hans Joachim Desserud <launchpad@xxxxxxxxxxxx>]

>The semantics of this code changes between release and debug builds and nobody will ever notice.
And I saw that in production code before.

I'd like to throw in a reminder that CppCheck is able to find these :) Scroll down to the latest report in bug 986611, search for assert and you'll see five places in the source code which does this. Not sure of their severity, but probably differences we shouldn't have.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_return_added/+merge/275638
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/ai_return_added.