This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl
- From: "pinskia at gcc dot gnu dot org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 1 Sep 2007 21:15:33 -0000
- Subject: [Bug c++/33208] Broken diagnostic: 'component_ref' not supported by dump_decl
- References: <bug-33208-1771@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla 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
> really the diagnostic machinery's business here to second guess
> a type error correction.
Actually "variable a" or "variable b" does not make sense here, we really want
to use the word lvalue instead of variable as shown by my other two testcases.
Yes I know it is not the diagnostic machinery's business here to second guess a
type error correction but I am saying we need to change the error message
fully. We should not be using "variable %qD" when we can get any kind of
lvalues here.
-- Pinski
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33208