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

Revision history for this message
SirVer (sirver) wrote :

I like this suggestion better too - it is how we deal with unreachable code everywhere else.

An even nicer solution would be if we had an UNREACHABLE macro that does behave exactly the same in every place. It could be implemented differently for each compiler, for example for clang there is http://clang.llvm.org/docs/LanguageExtensions.html#builtin-unreachable.

review: Approve

« Back to merge proposal