broulik added inline comments.

INLINE COMMENTS

> corona.cpp:378
> +        switch (d->immutability) {
> +        case Types::UserImmutable:
> +            action->setEnabled(false);

Why is this switch to explicitly stating all the cases?
The action is always visible and enabled only when Mutable

> corona.cpp:379
> +        case Types::UserImmutable:
> +            action->setEnabled(false);
> +            action->setVisible(true);

fall-through?

REPOSITORY
  R242 Plasma Framework (Library)

REVISION DETAIL
  https://phabricator.kde.org/D24263

To: mart, #plasma, #vdg
Cc: broulik, GB_2, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns

Reply via email to