widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/assert_removed into lp:widelands

To: mp+278269@xxxxxxxxxxxxxxxxxx
From: SirVer <SirVer@xxxxxx>
Date: Mon, 23 Nov 2015 06:17:12 -0000
In-reply-to: <20151122205033.11640.69332.launchpad@ackee.canonical.com>
Reply-to: mp+278269@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

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.


-- 
https://code.launchpad.net/~widelands-dev/widelands/assert_removed/+merge/278269
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/assert_removed.

References

[Merge] lp:~widelands-dev/widelands/assert_removed into lp:widelands
From: TiborB, 2015-11-22