← Back to team overview

widelands-dev team mailing list archive

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

 

qq: why not use    __attribute__ ((fallthrough)); [1] to get rid of the warnings? This is slightly ugly - so we could hide it in a macro:

#define FALLTHROUGH_INTENDED __attribute__ ((fallthrough)) 

This would make it explicit that we did intend for this to fall through. It should also be compatible across all compilers.

[1] https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/
-- 
https://code.launchpad.net/~widelands-dev/widelands/gcc7/+merge/323576
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/gcc7.


References