KDE/kdebase/workspace/libs/plasma

[Alex Merry]

On Monday 28 April 2008 10:53:46 Alex Merry wrote:
> On Saturday 26 April 2008 16:52:49 Marco Martin wrote:
> > SVN commit 801425 by mart:
> >
> > API review:
> > isContainment(), setIsContainment() -> actAsContainment(),
> > setActAsContainment()
> >  set is protected
>
> It's a minor point, but wouldn't "actsAsContainment()" be better than
> "actAsContainment()"?  "actAsContainment()" sounds like the method will do
> something (cause it to act as a containment), whereas "actsAsContainment()"
> sounds like it will return a bool saying whether it does or not.

OK, Aaron reverted this anyway, so it's a moot point.

Alex


-- 
KDE: http://www.kde.org
Ubuntu/Kubuntu: http://www.ubuntu.org http://www.kubuntu.org

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/panel-devel/attachments/20080428/3c985da5/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080428/3c985da5/attachment.pgp