[Bug c++/53184] Unnecessary anonymous namespace warnings

2015-09-09 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 Paolo Carlini changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/53184] Unnecessary anonymous namespace warnings

2015-09-09 Thread paolo at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #11 from paolo at gcc dot gnu.org --- Author: paolo Date: Wed Sep 9 16:48:13 2015 New Revision: 227593 URL: https://gcc.gnu.org/viewcvs?rev=227593&root=gcc&view=rev Log: 2015-09-09 Paolo Carlini PR c++/53184 * do

[Bug c++/53184] Unnecessary anonymous namespace warnings

2015-08-19 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 Paolo Carlini changed: What|Removed |Added CC||blue.dragan+gcc at gmail dot com --- Co

[Bug c++/53184] Unnecessary anonymous namespace warnings

2015-04-27 Thread paolo.carlini at oracle dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 Paolo Carlini changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug c++/53184] Unnecessary anonymous namespace warnings

2013-06-17 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #9 from Paolo Carlini --- ... and of course a clearer wording for the warning itself.

[Bug c++/53184] Unnecessary anonymous namespace warnings

2013-06-17 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #8 from Paolo Carlini --- Somebody could suggest an appropriate name for the warning. Then a patchlet would be easy to do and also easy to approve I think (naming warnings is in general a sensible idea)

[Bug c++/53184] Unnecessary anonymous namespace warnings

2013-06-17 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #7 from Jonathan Wakely --- I've just come across a case where an option to disable that warning would be handy: We have a foo.cc file defining some classes in an anon namespace. A unit test does #include "foo.cc" so it can test the im

[Bug c++/53184] Unnecessary anonymous namespace warnings

2013-06-15 Thread zeratul976 at hotmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #6 from Nathan Ridge --- (In reply to Jason Merrill from comment #4) > (In reply to comment #3) > > Because then the anonymous class has the name "Foo" for linkage purposes, > > and > > has external linkage. When Foo referes to the c

[Bug c++/53184] Unnecessary anonymous namespace warnings

2013-06-15 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 Paolo Carlini changed: What|Removed |Added CC||zeratul976 at hotmail dot com --- Comment

[Bug c++/53184] Unnecessary anonymous namespace warnings

2012-05-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #4 from Jason Merrill 2012-05-02 17:23:09 UTC --- (In reply to comment #3) > Because then the anonymous class has the name "Foo" for linkage purposes, and > has external linkage. When Foo referes to the const or volatile qualified >

[Bug c++/53184] Unnecessary anonymous namespace warnings

2012-05-02 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 Jonathan Wakely changed: What|Removed |Added CC||jason at gcc dot gnu.org --- Comment #3

[Bug c++/53184] Unnecessary anonymous namespace warnings

2012-05-01 Thread orgads at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #2 from Orgad Shaneh 2012-05-02 06:56:28 UTC --- Anonymous types shouldn't produce any warnings. They are very commonly used in headers needed for both C and C++ projects. The warning is meant for types inside anonymous namespaces, wh

[Bug c++/53184] Unnecessary anonymous namespace warnings

2012-05-01 Thread pinskia at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53184 --- Comment #1 from Andrew Pinski 2012-05-02 06:25:50 UTC --- >foo.cpp:5:16: warning: 'Bar' has a field 'Bar::foo' whose type uses the anonymous namespace [enabled by default] The warning just needs to be worded better but otherwise it is correc