[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2023-09-26 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Andrew Pinski changed: What|Removed |Added CC||Hi-Angel at yandex dot ru --- Comment

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2021-10-26 Thread egallager at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Eric Gallager changed: What|Removed |Added CC||antmak.pub at gmail dot com,

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2019-05-14 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Eric Gallager changed: What|Removed |Added CC||gcc-bugs at engestrom dot ch ---

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2018-07-05 Thread antmak.pub at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #18 from Anton Maklakov --- Thank you for the answer! I've got. Using the hint after #endif is not pretty good, by the logic. But this is easier.

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2018-07-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #17 from Jakub Jelinek --- (In reply to Anton Maklakov from comment #16) > Hi, > > I have the same problem: > > > int main() { > int s = 1; > > switch (s) { > case 2: > s = 2; > #if B >

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2018-07-05 Thread antmak.pub at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Anton Maklakov changed: What|Removed |Added CC||antmak.pub at gmail dot com ---

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2017-09-28 Thread marc.mutz at kdab dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #15 from Marc Mutz --- (In reply to Jakub Jelinek from comment #13) > (In reply to Marc Mutz from comment #12) > > Is replacing a matching comment with __attribute__(fallthrough)) so > > complicated as to make this a wontfix? > > It

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2017-09-28 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Eric Gallager changed: What|Removed |Added CC||sgunderson at bigfoot dot com ---

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #13 from Jakub Jelinek --- (In reply to Marc Mutz from comment #12) > Is replacing a matching comment with __attribute__(fallthrough)) so > complicated as to make this a wontfix? It is not really possible.

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-12 Thread marc.mutz at kdab dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #12 from Marc Mutz --- Is replacing a matching comment with __attribute__(fallthrough)) so complicated as to make this a wontfix?

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #11 from Jakub Jelinek --- (In reply to Markus Trippelsdorf from comment #10) > The testcase from PR77955: > > markus@x4 /tmp % cat fall.c > void bar(int); > > void foo(int i) { > switch (i) { > case 1: { > bar(1); > //

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-12 Thread trippels at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #10 from Markus Trippelsdorf --- The testcase from PR77955: markus@x4 /tmp % cat fall.c void bar(int); void foo(int i) { switch (i) { case 1: { bar(1); // fall-through } case 2: bar(2); default: break; }

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-12 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Marek Polacek changed: What|Removed |Added CC||trippels at gcc dot gnu.org --- Comment

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-09 Thread marc.mutz at kdab dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Marc Mutz changed: What|Removed |Added CC||marc.mutz at kdab dot com --- Comment #8

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-05 Thread jim at meyering dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #7 from jim at meyering dot net --- Thanks for investigating. I already pushed a workaround for gnulib's vasnprintf problem (http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=e63f5eb55570a1ea3e51ce47df33689785e085c1). I may just

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #6 from Jakub Jelinek --- (In reply to Marek Polacek from comment #5) > Yeah, I think this one's WONTFIX. Well, in theory we could for the comments that satisfy fallthrough_comment_p create CPP_COMMENT token and insert it, even

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-05 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #5 from Marek Polacek --- Yeah, I think this one's WONTFIX.

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-05 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #4

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-04 Thread amodra at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Alan Modra changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-03 Thread jim at meyering dot net
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 --- Comment #2 from jim at meyering dot net --- Oops. That must have been the "worked-around" version. Swap the #undef and fallthrough comment to repro: int foo (int x) { switch (x) { case 1: x = 3; /* fallthrough */ #undef

[Bug c/77817] -Wimplicit-fallthrough: cpp directive renders FALLTHRU comment ineffective

2016-10-03 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77817 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment