apol added inline comments.

INLINE COMMENTS

> plasmawindowmanagement_interface.cpp:386
> +{
> +    m_icon = icon;
> +    setThemedIconName(m_icon.name());

maybe worth adding a `if (m_icon == icon) return;`

REPOSITORY
  rKWAYLAND KWayland

BRANCH
  plasma-window-icon

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

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

To: graesslin, #plasma_on_wayland, hein, roberts
Cc: apol, roberts, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas

Reply via email to