[Kde-pim] Review Request: MessageViewer: fix warnings about overloaded virtuals being hidden
Kevin Krammer
krammer at kde.org
Sat Dec 29 12:54:36 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108001/
-----------------------------------------------------------
Review request for KDEPIM and Laurent Montel.
Description
-------
When building messageviewer and related plugins, the compiler warns about virtual methods defined in base classes or interface as being hidden by the subclasses.
In the BodyFormatter cases it should be harmless since the hidden overload just delegates to the reimplemented one.
In the accessibility case it seems to be a two functions from different interface being considered overloads. One is from QAccessible v1 and one is from QAccessible v2. might be necessary to check whether the v1 one can be removed
Diffs
-----
messageviewer/a11y/accessiblemailwebview.h 5558551
messageviewer/bodypartformatter.cpp 45ba488
plugins/messageviewer/bodypartformatter/application_ms-tnef.cpp 263f467
plugins/messageviewer/bodypartformatter/text_xdiff.cpp 4660668
Diff: http://git.reviewboard.kde.org/r/108001/diff/
Testing
-------
Thanks,
Kevin Krammer
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
More information about the kde-pim
mailing list