[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[redi at gcc dot gnu.org]

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

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> 2010-11-01 11:22:26 UTC ---
This would also be solved by the "override control" features coming in C++0x

c.f. PR 17920, PR 31397, PR 36848

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[loose at astron dot nl]

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

--- Comment #3 from loose at astron dot nl 2010-11-01 12:29:27 UTC ---
As far as I know attributes were removed from the C0x Final Draft.
Besides, if GCC were to provide a compiler flag to warn for this, you would
also be able to catch potential bugs in "old" code, or in code where the author
was not explicit about overrides.

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[redi at gcc dot gnu.org]

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

--- Comment #4 from Jonathan Wakely <redi at gcc dot gnu.org> 2010-11-01 12:36:45 UTC ---
Attributes were still in the FCD, and are still in the current draft, but might
be removed.  Override control may not be done by attributes, but will be
present in C++0x in some form, see
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2010/n3151.html and
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2010/n3163.pdf for the
latest proposals.

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[redi at gcc dot gnu.org]

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

--- Comment #5 from Jonathan Wakely <redi at gcc dot gnu.org> 2010-11-01 12:39:48 UTC ---
also, GNU-style attributes are not being removed from GCC, even if C++0x
attributes get dropped

my point was just that there are several similar bugs and that whatever it
looks like, the final implementation of C++0x override control is likely to
influence this enhancement request.

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[loose at astron dot nl]

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

--- Comment #6 from loose at astron dot nl 2010-11-01 12:54:25 UTC ---
OK, fair enough.

However, I reported this "bug", because I would like GCC to produce a warning
when hiding a non-virtual method in *existing* code; an option similar to
-Woverloaded-virtual.

New code could of course be written, using either C0x attributes or GCC
attributes. But that won't solve the issue for C98-compliant code.

I agree that it's preferable to remove duplicate bugs. However, I'm not sure
the bugs you mention are real duplicates, because at least some of them
explicitly refer to overriding (virtual) functions, whereas I'm referring to
overloading (non-virtual) functions.

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[redi at gcc dot gnu.org]

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

--- Comment #7 from Jonathan Wakely <redi at gcc dot gnu.org> 2010-11-01 13:01:43 UTC ---
Yup, that's why I haven't closed this as a duplicate.  

The C++0x override control works for virtual and non-virtual names, so is
relevant to this PR *and* the others.  Noting and linking to similar PRs helps
people looking at this in future.

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[jason at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=20710

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |EisahLee at gmx dot de

--- Comment #11 from Jason Merrill <jason at gcc dot gnu.org> ---
*** Bug 67345 has been marked as a duplicate of this bug. ***

[Bug c++/20710] g++ should warn when hiding non-virtual method in base class

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2005-07-02 01:00 -------
Confirmed.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |diagnostic
   Last reconfirmed|0000-00-00 00:00:00         |2005-07-02 01:00:21
               date|                            |


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