[Bug c++/16604] Bad diagnostic

[Bug c++/16604] Bad diagnostic

[igodard at pacbell dot net]

------- Additional Comments From igodard at pacbell dot net  2004-07-17 08:29 -------
Created an attachment (id=6768)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=6768&action=view)
Compiler output (-v -save-temps)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] New: Bad diagnostic

[igodard at pacbell dot net]

First disgnostic (of many :-(  complains:/home/ivan/ootbc/common/include/display.hh:480: error: using-declaration for non
-member at class scope

However, there's no using declaration there. The actual error is that the
sizeof argument should have been "ch", not "Ch". The diagnostic should have reported "Ch" as undefined.

Ivan

-- 
           Summary: Bad diagnostic
           Product: gcc
           Version: 3.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: igodard at pacbell dot net
                CC: gcc-bugs at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] Bad diagnostic

[igodard at pacbell dot net]

------- Additional Comments From igodard at pacbell dot net  2004-07-17 08:32 -------
Created an attachment (id=6769)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=6769&action=view)
Source code (-save-temps)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] Bad diagnostic

[igodard at pacbell dot net]

------- Additional Comments From igodard at pacbell dot net  2004-07-17 08:39 -------
Correction: an additional bug was that "Breaker" should have been "breaker", and the diagnostic should have complained that both Breaker and Ch were undefined. I'm guessing that the appearance of "<namespace>::<identifier>" is considered a using-declaration internally, but syntactically it's a qualified name. A diagnostic that talks about using-declarations is confusing.

Ivan

-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] Bad diagnostic

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-07-17 09:06 -------
I can confirm this but I do not have a smaller testcase yet.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |minor
           Keywords|                            |diagnostic


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] Bad diagnostic

[bangerth at dealii dot org]

------- Additional Comments From bangerth at dealii dot org  2004-07-19 13:36 -------
The real bug is that you misspelled Hide::Breaker: it is only declared 
with the lowercase name Hide::breaker. 
 
This is the testcase: 
------------------- 
namespace NS {} 
 
class X { 
    NS::NonExistent x; 
}; 
------------------- 
 
g/x> /home/bangerth/bin/gcc-3.4-pre/bin/c++ -c -w x.cc 
x.cc:4: error: using-declaration for non-member at class scope 
x.cc:4: error: expected `;' before "x" 
 
This is already fixed in mainline, though: 
 
g/x> /home/bangerth/bin/gcc-3.5-pre/bin/c++ -c x.cc 
x.cc:4: error: `NonExistent' in namespace `NS' does not name a type 
 
W. 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

Re: [Bug c++/16604] Bad diagnostic

[Gabriel Dos Reis]

bangerth at dealii dot org:
>
> ------- Additional Comments From bangerth at dealii dot org  2004-07-19
> 13:36 -------
> The real bug is that you misspelled Hide::Breaker: it is only declared
> with the lowercase name Hide::breaker.
>
> This is the testcase:
> -------------------
> namespace NS {}
>
> class X {
>     NS::NonExistent x;
> };
> -------------------
>
> g/x> /home/bangerth/bin/gcc-3.4-pre/bin/c++ -c -w x.cc
> x.cc:4: error: using-declaration for non-member at class scope
> x.cc:4: error: expected `;' before "x"
>
> This is already fixed in mainline, though:
>
> g/x> /home/bangerth/bin/gcc-3.5-pre/bin/c++ -c x.cc
> x.cc:4: error: `NonExistent' in namespace `NS' does not name a type

Indeed.  That is an improvement.  But, it still leaves the impression
that 'NonExistent' may designate something else, contrary to
what its spelling suggests :-)

We can tell the difference by looking at BINDING_VALUE and
BINDING_TYPE of the identifier.  Any taker? :-)

[Bug c++/16604] Bad diagnostic

[gdr at acm dot org]

------- Additional Comments From gdr at acm dot org  2004-07-19 13:40 -------
Subject: Re:  Bad diagnostic

bangerth at dealii dot org:
>
> ------- Additional Comments From bangerth at dealii dot org  2004-07-19
> 13:36 -------
> The real bug is that you misspelled Hide::Breaker: it is only declared
> with the lowercase name Hide::breaker.
>
> This is the testcase:
> -------------------
> namespace NS {}
>
> class X {
>     NS::NonExistent x;
> };
> -------------------
>
> g/x> /home/bangerth/bin/gcc-3.4-pre/bin/c++ -c -w x.cc
> x.cc:4: error: using-declaration for non-member at class scope
> x.cc:4: error: expected `;' before "x"
>
> This is already fixed in mainline, though:
>
> g/x> /home/bangerth/bin/gcc-3.5-pre/bin/c++ -c x.cc
> x.cc:4: error: `NonExistent' in namespace `NS' does not name a type

Indeed.  That is an improvement.  But, it still leaves the impression
that 'NonExistent' may designate something else, contrary to
what its spelling suggests :-)

We can tell the difference by looking at BINDING_VALUE and
BINDING_TYPE of the identifier.  Any taker? :-)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604

[Bug c++/16604] Bad diagnostic

[pinskia at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |3.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16604