{"action":"created","issue":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8603","repository_url":"https://api.github.com/repos/RIOT-OS/RIOT","labels_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8603/labels{/name}","comments_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8603/comments","events_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8603/events","html_url":"https://github.com/RIOT-OS/RIOT/pull/8603","id":298621678,"number":8603,"title":"make: always set -Wno-implicit-fallthrough","user":{"login":"kaspar030","id":4679640,"avatar_url":"https://avatars0.githubusercontent.com/u/4679640?v=4","gravatar_id":"","url":"https://api.github.com/users/kaspar030","html_url":"https://github.com/kaspar030","followers_url":"https://api.github.com/users/kaspar030/followers","following_url":"https://api.github.com/users/kaspar030/following{/other_user}","gists_url":"https://api.github.com/users/kaspar030/gists{/gist_id}","starred_url":"https://api.github.com/users/kaspar030/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kaspar030/subscriptions","organizations_url":"https://api.github.com/users/kaspar030/orgs","repos_url":"https://api.github.com/users/kaspar030/repos","events_url":"https://api.github.com/users/kaspar030/events{/privacy}","received_events_url":"https://api.github.com/users/kaspar030/received_events","type":"User","site_admin":false},"labels":[{"id":234348609,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/CI","name":"CI","color":"fad8c7","default":false},{"id":214816571,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/Ready%20for%20CI%20build","name":"Ready for CI build","color":"bfdadc","default":false},{"id":97338598,"url":"https://api.github.com/repos/RIOT-OS/RIOT/labels/make","name":"make","color":"fad8c7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-02-20T14:50:38Z","updated_at":"2018-02-20T15:16:17Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/pulls/8603","html_url":"https://github.com/RIOT-OS/RIOT/pull/8603","diff_url":"https://github.com/RIOT-OS/RIOT/pull/8603.diff","patch_url":"https://github.com/RIOT-OS/RIOT/pull/8603.patch"},"body":"### Contribution description\r\n\r\ngcc 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.\r\n\r\nThere have been numerous PR's fixing remaining fallthrough warnings, mostly in packages. Still, many are left.\r\n\r\nIdeally 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...\r\n\r\nI propose just disabling the warning for now, globally, as this PR does."},"comment":{"url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/comments/367009694","html_url":"https://github.com/RIOT-OS/RIOT/pull/8603#issuecomment-367009694","issue_url":"https://api.github.com/repos/RIOT-OS/RIOT/issues/8603","id":367009694,"user":{"login":"gebart","id":317506,"avatar_url":"https://avatars1.githubusercontent.com/u/317506?v=4","gravatar_id":"","url":"https://api.github.com/users/gebart","html_url":"https://github.com/gebart","followers_url":"https://api.github.com/users/gebart/followers","following_url":"https://api.github.com/users/gebart/following{/other_user}","gists_url":"https://api.github.com/users/gebart/gists{/gist_id}","starred_url":"https://api.github.com/users/gebart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gebart/subscriptions","organizations_url":"https://api.github.com/users/gebart/orgs","repos_url":"https://api.github.com/users/gebart/repos","events_url":"https://api.github.com/users/gebart/events{/privacy}","received_events_url":"https://api.github.com/users/gebart/received_events","type":"User","site_admin":false},"created_at":"2018-02-20T15:16:17Z","updated_at":"2018-02-20T15:16:17Z","author_association":"MEMBER","body":"Can't we make a push on the fixing PRs instead? #8265 \r\nAt least not hide the warnings, it would be better to just disable the error: `-Wno-error=implicit-fallthrough`"}}
{"actor":{"display_login":"gebart"}}