[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-02 Thread paolo at gcc dot gnu dot org
--- Comment #10 from paolo at gcc dot gnu dot org 2007-09-02 13:02 --- Subject: Bug 33208 Author: paolo Date: Sun Sep 2 13:02:31 2007 New Revision: 128025 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=128025 Log: /cp 2007-09-02 Paolo Carlini [EMAIL PROTECTED] PR

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-02 Thread pcarlini at suse dot de
--- Comment #11 from pcarlini at suse dot de 2007-09-02 13:03 --- Fixed. -- pcarlini at suse dot de changed: What|Removed |Added Status|ASSIGNED

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pinskia at gcc dot gnu dot org
--- Comment #1 from pinskia at gcc dot gnu dot org 2007-09-01 20:03 --- Confirmed. The problem is obvious: error (invalid use of %--% on bool variable %qD, arg); %qD should be %qE. -- pinskia at gcc dot gnu dot org changed: What|Removed

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pcarlini at suse dot de
--- Comment #2 from pcarlini at suse dot de 2007-09-01 20:52 --- But do we really want 'a.A::b' ?!? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33208

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pinskia at gcc dot gnu dot org
--- Comment #3 from pinskia at gcc dot gnu dot org 2007-09-01 21:01 --- (In reply to comment #2) But do we really want 'a.A::b' ?!? Well the error message needs rewording also. Something like: invalid use of %--% on bool lvalue %qE --

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread gdr at cs dot tamu dot edu
--- Comment #4 from gdr at cs dot tamu dot edu 2007-09-01 21:07 --- Subject: Re: Broken diagnostic: 'component_ref' not supported by dump_decl pcarlini at suse dot de [EMAIL PROTECTED] writes: | But do we really want 'a.A::b' ?!? No, we don't. The format specific is OK -- e.g. it

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pcarlini at suse dot de
--- Comment #5 from pcarlini at suse dot de 2007-09-01 21:11 --- Thanks Gaby, let's see what I can do... -- pcarlini at suse dot de changed: What|Removed |Added

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pinskia at gcc dot gnu dot org
--- Comment #6 from pinskia at gcc dot gnu dot org 2007-09-01 21:12 --- Another testcase: void f(bool *b) { (*b)--; } And another one: bool g(void); void f(bool *b) { g()--; } So variable in the error message does not make sense at all. We want to use lvalue instead. --

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pinskia at gcc dot gnu dot org
--- Comment #7 from pinskia at gcc dot gnu dot org 2007-09-01 21:15 --- (In reply to comment #4) | But do we really want 'a.A::b' ?!? No, we don't. The format specific is OK -- e.g. it should be %qD. However, the caller of error() should make sure it gives a _DECL. It isn't

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread gdr at cs dot tamu dot edu
--- Comment #8 from gdr at cs dot tamu dot edu 2007-09-01 21:59 --- Subject: Re: Broken diagnostic: 'component_ref' not supported by dump_decl pinskia at gcc dot gnu dot org [EMAIL PROTECTED] writes: | Another testcase: | void f(bool *b) | { | (*b)--; | } | | And another one: |

[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl

2007-09-01 Thread pcarlini at suse dot de
--- Comment #9 from pcarlini at suse dot de 2007-09-01 22:14 --- (In reply to comment #8) Paolo, what about error (invalid use of Boolean expression as operand to %operator--%) ? Cetainly works for me... -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33208