--- 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
--- 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
--- 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
--- 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
--- 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
--
--- 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
--- 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
--- 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.
--
--- 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
--- 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:
|
--- 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
11 matches
Mail list logo