[Differential] [Changed Subscribers] D2630: Support highlighting windows through EffectsHandlerImpl

[sebas (Sebastian Kügler)]

sebas added inline comments.

INLINE COMMENTS

> effects.h:239
>  
> +    void highlightWindows(const QVector<EffectWindow *> &windows);
> +

EffectWindow* ?

> highlightwindow.cpp:261
>  
> +void HighlightWindowEffect::highlightWindows(const QVector<KWin::EffectWindow *> &windows)
> +{

whitespace

> highlightwindow.h:58
>  
> +    void highlightWindows(const QVector<KWin::EffectWindow *> &windows);
> +

EffectWindow* (no space)

REPOSITORY
  rKWIN KWin

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland
Cc: sebas, luebking, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160912/1936e9be/attachment.html>