On Mon, Mar 23, 2009 at 4:03 PM, Aaron J. Seigo <ase...@kde.org> wrote:
>        bool hidden() const;
>        void setHidden(HideState state);

Asymmetry is evil. If set takes a HideState then get should return it.
Having another method that is a more general helper is ok though.

bool hidden() const;

void setHideState(HideState state);
HideState hideState() const;

This has the advantage of being clearer and also being possible to
make a Q_PROPERTY.

Cheers

Rich.
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to