[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2015-02-03 Thread dodji at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #19 from Dodji Seketeli --- (In reply to Tom Tromey from comment #16) > I've tripped across this enough that I've actually filed dups twice now. > > I think it would be best to change the ordering here. > That is, the initial error o

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2015-02-01 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #18 from Manuel López-Ibáñez --- (In reply to Jonathan Wakely from comment #17) > It would be less of a pain if -Wsystem-headers caused both locations to be > printed, but it doesn't, so sometimes the only option is to dump the > prep

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2015-02-01 Thread redi at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #17 from Jonathan Wakely --- It would be less of a pain if -Wsystem-headers caused both locations to be printed, but it doesn't, so sometimes the only option is to dump the preprocessed source without line markers and then compile tha

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2014-07-15 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #16 from Tom Tromey --- I've tripped across this enough that I've actually filed dups twice now. I think it would be best to change the ordering here. That is, the initial error ought to generally be the location of the outermost exp

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2014-07-15 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #15 from Tom Tromey --- *** Bug 61803 has been marked as a duplicate of this bug. ***

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2014-05-13 Thread tromey at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 Tom Tromey changed: What|Removed |Added CC||tromey at gcc dot gnu.org --- Comment #14 f

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2013-12-13 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 Manuel López-Ibáñez changed: What|Removed |Added CC||doront at mellanox dot com --- Comm

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2013-11-01 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #12 from Manuel López-Ibáñez --- PR52962 is another case where it would make more sense to give the error on the macro expansion location rather than on the macro definition.

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2013-11-01 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 Manuel López-Ibáñez changed: What|Removed |Added CC||michael at talamasca dot ocis.net

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-19 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #10 from dodji at seketeli dot org 2012-11-19 17:18:00 UTC --- "manu at gcc dot gnu.org" a écrit: > The idea seems good. It would also handle comment #4 testcase. Yeah, and I think it would be a step in the direction of printing r

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-19 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #9 from dodji at seketeli dot org 2012-11-19 17:05:57 UTC --- "manu at gcc dot gnu.org" a écrit: > Hum, I am not sure why the macro unwinder avoids unwinding if the > macro comes from a system-header. If a warning message comes fro

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-19 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #8 from Manuel López-Ibáñez 2012-11-19 16:50:34 UTC --- (In reply to comment #7) > Would that make sense in the grand scheme of things? The idea seems good. It would also handle comment #4 testcase. However, I am not sure how you wo

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-19 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #7 from dodji at seketeli dot org 2012-11-19 16:34:11 UTC --- "manu at gcc dot gnu.org" a écrit: > On the other hand, let's consider: > pr55252.c: > > #define bar 256 > #include "pr55252.h" > > pr55252.h: > > #pragma GCC system_hea

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-19 Thread dodji at seketeli dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #6 from dodji at seketeli dot org 2012-11-19 16:17:20 UTC --- > I think this his how the macro expansion was designed to work: It > shows the location of the token that triggered the error. Yes. And there are cases where the GCC wa

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-10 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #5 from Jonathan Wakely 2012-11-10 23:20:36 UTC --- (In reply to comment #4) > On the other hand, this is a very contrived testcase. I > wouldn't expect in normal code that the expansion point to be in a > system-header and th

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-10 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #4 from Manuel López-Ibáñez 2012-11-10 23:10:32 UTC --- On the other hand, let's consider: pr55252.c: #define bar 256 #include "pr55252.h" pr55252.h: #pragma GCC system_header signed char foo = bar; In this case, I would expect th

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-10 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 --- Comment #3 from Manuel López-Ibáñez 2012-11-10 22:58:16 UTC --- Hum, I am not sure why the macro unwinder avoids unwinding if the macro comes from a system-header. If a warning message comes from a system-header, then it should have been sup

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-09 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 Manuel López-Ibáñez changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c++/55252] Caret diagnostic doesn't show useful location when macro clashes with name in system header

2012-11-09 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55252 Jonathan Wakely changed: What|Removed |Added CC||manu at gcc dot gnu.org --- C