On Mon, Mar 23, 2009 at 4:03 PM, Aaron J. Seigo <[email protected]> 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
[email protected]
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to