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