[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