gcc -Woverloaded-virtual
André Wöbbeking
Woebbeking at kde.org
Sat Nov 17 14:48:40 GMT 2007
On Saturday 17 November 2007, Matthias Kretz wrote:
> On Saturday 17 November 2007, André Wöbbeking wrote:
> >
> > It can be very handy but there're too many false positives, e.g. if
> > virtual void A::f(int) also exists. IMO too many to make this a
> > default option.
>
> It's not really a false positive as B::f() still gets hidden, no?
If you mean A::f(), yes.
> I agree that you sometimes don't care, e.g. if the implementation
> class is only used casted down to the interface class.
This is the case that matters. If you try to call B::f() you get a
compile error.
> Perhaps we
> want to make it a configure switch then? Let's see how noisy it gets
> first I'd say...
OK
More information about the kde-core-devel
mailing list