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