>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. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
