ivan added a comment.

  Very nice, thanks. If you want I can accept this (after the `Q_UNUSED` 
change) and I'll do the UI patch separately.

INLINE COMMENTS

> service.cpp:53
> +    {
> +        (void)enabled;
> +    }

You should use `Q_UNUSED` instead.

REPOSITORY
  R845 Plasma Vault

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

To: asturmlechner, #plasma, ivan
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to