--- 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 #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=gcc&view=rev&rev=128025
Log:
/cp
2007-09-02 Paolo Carlini <[EMAIL PROTECTED]>
PR c
--- 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 %")
>
> ?
Cetainly works for me...
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33208
--- 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 #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 #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.
--
htt
--- 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
A
--- 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. i
--- 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
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?i
--- 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 #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
11 matches
Mail list logo
