Review Request: Introduce two new KCrash methods: void setLaunchDrKonqi(bool) and bool launchDrKonqi().

Chani chanika at gmail.com
Sun Oct 25 23:33:37 GMT 2009


> >
> > If you call it "enable", it's an action. The opposite is "disable", not a
> > true/false parameter.
> 
> /me is getting confused with all this...
> 
> Ok, I agree that launchDrKonqi() is not a good name for a getter. How
> about these:
> 
> bool isDrKonqiEnabled()
> void setDrKonqiEnabled(bool)
> 

that's the conclusion we came to, accidentally off-list. :)

-- 
This message brought to you by eevil bananas and the number 3.
www.chani3.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20091025/99b256a0/attachment.sig>


More information about the kde-core-devel mailing list