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

[Chani]

On October 25, 2009 11:45:02 Thiago Macieira wrote:
> Em Domingo 25. Outubro 2009, às 17.33.09, George Kiagiadakis escreveu:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviewboard.kde.org/r/1970/
> > -----------------------------------------------------------
> >
> > Review request for kdelibs.
> >
> >
> > Summary
> > -------
> >
> > This patch introduces two new methods in the KCrash namespace: void
> >  setLaunchDrKonqi(bool) and bool launchDrKonqi(). The reason for
> 
> The name "launchDrKonqi" is a bad one. "Launch" is a verb in the
>  imperative, so "launchDrKonqi" doesn't look like a setter, but an action.
> 
> I recommend using "launchesDrKonqi" instead.
> 
> Also, for boolean properties, prefixing the getter with "is" is
>  recommended, but then you need an adjective as the core of the property
>  name (as in "isDrKonqiLaunchEnabled"). If you can't come up with a good
>  name, don't use "is", it's not a problem.
> 

enableDrKonqui and isDrKonquiEnabled?

hrm, we should probably be replying on reviewboard instead of by mail... darn 
web things.

-- 
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/eb71dc57/attachment.sig>