PlasmaKPart moved to kdereview

David Faure faure at kde.org
Tue Aug 17 21:21:27 CEST 2010


On Friday 13 August 2010, Richard Moore wrote:
> bool openFile() { return true; }; inline method and dubious

This one is actually OK in ReadOnlyPart derived classes.
If you reimplement openUrl(), then openFile() can be just "return false" (the 
docs say to return false, not true, but it really doesn't matter, it's never 
called).

Years after the fact, I can see that openFile() shouldn't have been a pure 
virtual, but a simple " { return false; }" default implementation. I guess I 
can make that change now, actually, this should be BC/SC. Doing that now.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).


More information about the Plasma-devel mailing list