romangg requested changes to this revision.
romangg added inline comments.
This revision now requires changes to proceed.
Restricted Application edited projects, added KWin; removed Plasma.

INLINE COMMENTS

> wayland_cursor_theme.cpp:69
>                                     size, m_shm->shm());
> -    emit themeChanged();
> +    if (theme) {
> +        if (!m_theme) {

Only do if theme really changes (`theme && m_theme != theme`).

> wayland_cursor_theme.cpp:79
> +    }
>  }
>  

On `!theme`: Add debug output that `theme`could not be loaded.

REPOSITORY
  R108 KWin

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

To: graesslin, #kwin, #plasma, davidedmundson, romangg
Cc: romangg, plasma-devel, kwin, iodelay, bwowk, ZrenBot, lesliezhai, 
ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart

Reply via email to