[riot-notifications] [RIOT-OS/RIOT] make: always set -Wno-implicit-fallthrough (#8603)

Kaspar Schleiser notifications at github.com
Tue Feb 20 15:50:45 CET 2018


### Contribution description

gcc 7 introduced a new warning for uncommented implicit switch/case fallthroughs. As we enable all plus extra warnings, gcc 7 stumbles over a couple of those in our codebase.

There have been numerous PR's fixing remaining fallthrough warnings, mostly in packages. Still, many are left.

Ideally we'd fix them all before CI is switching to gcc 7. Unfortunately, somehow the ppa's used within the RIOT Dockerfile now default to a newer gcc7 arm toolchain, thus forcing our hand...

I propose just disabling the warning for now, globally, as this PR does.
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/8603

-- Commit Summary --

  * make: always set -Wno-implicit-fallthrough

-- File Changes --

    M Makefile.include (3)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/8603.patch
https://github.com/RIOT-OS/RIOT/pull/8603.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/8603
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20180220/9cee0bef/attachment.html>


More information about the notifications mailing list