[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #13 from Paolo Carlini --- I added c++/57880 to the svn CL. Sorry, I'm not going to reference by hand each and every patch I commit (svn should do it automatically, send a message to Bugzilla, as it used to).

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #12 from David Binderman --- (In reply to Paolo Carlini from comment #11) > Fixed for 4.9.0. Good news. Worth referencing the patch that fixed it ?

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 Paolo Carlini changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 Paolo Carlini changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 Paolo Carlini changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #8 from Paolo Carlini --- If we have a straightforward explanation for why the very weird code isn't causing problems, I think the fix almost qualifies as obvious, if properly tested. Do we? Does it only amount to a small memory leak o

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #7 from Jonathan Wakely --- Yes, or at gcc.gnu.org

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #6 from David Binderman --- (In reply to Paolo Carlini from comment #4) > For a quicker review, I would recommend CC-ing Jason and adding [C++ Patch] > to the subject line. I'm not on the inner wheel for C++ patches, so I am guessing

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #5 from David Binderman --- Created attachment 30545 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30545&action=edit source code patch This patch seems to shut up cppcheck. I did a bootstrap and that seemed ok. It seems good t

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #3 from David Binderman --- (In reply to Paolo Carlini from comment #2) > Ed? He's had eleven days to think about it and said nothing. I'd be happy to supply the obvious patch.

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #4 from Paolo Carlini --- Maybe he is simply traveling or unable to read email, don't you think? Still, if you want to send a patch to the mailing list, of course it's fine. For a quicker review, I would recommend CC-ing Jason and add

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 --- Comment #2 from Paolo Carlini --- Ed?

[Bug c++/57880] cp/parser.c: 6 * missing break ?

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57880 Paolo Carlini changed: What|Removed |Added CC||emsr at gcc dot gnu.org Componen