D7871: Properly update the visible (icon) name when the caption changes

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> client.cpp:1449
>          if (!s[i].isPrint())
>              s[i] = QChar(u' ');
> +    const bool changed = (s != cap_normal);

(Oh! I just spotted why that guy on reddit back a while had KWin add loads of spaces to the title of its windows when it contained Unicode control characters that didn't show up in Task Manager but KWin replaced them with spaces)

REPOSITORY
  R108 KWin

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

To: graesslin, #kwin, #plasma, hein
Cc: broulik, plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170918/978d7134/attachment.html>