Review Request: document KUniqueApplication::loadCommandLineOptionsForNewInstance and implement stub
Michael Jansen
kde at michael-jansen.biz
Sat Jun 12 18:53:50 BST 2010
>
> trunk/KDE/kdelibs/kdeui/kernel/kuniqueapplication.h
> <http://reviewboard.kde.org/r/4278/#comment5689>
>
> I don't think we should document it. The comment in the .cpp file says
> it is there to support Kontact, and I fail to see a reason why other
> applications would need this kludge.
>
> At least remove the "virtual" from the comment.
It is used by skansuite too. So it is already spreading. I'm for documenting it but the last word
should be by david.
Mike
More information about the kde-core-devel
mailing list