<table><tr><td style="">dvratil added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27675">View Revision</a></tr></table><br /><div><div><p>The system makes sense on X11 where you get something like <tt style="background: #ebebeb; font-size: 13px;">HDMI-1 (Sony GigaTV-12345)</tt> so you see which device it is and what physical output it's connected to. Since KWin does not use to output names but only the names of the actually connected devices, the needed behavior is different.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D27675#inline-157270">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ahmadsamir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">utils.cpp:64</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">If I am reading this correctly, you could keep the old code and use simplified():<br />
const QString outName = output->name().simplified();</p>

<p style="padding: 0; margin: 8px;">then later on:<br />
name = name.simplified();<br />
if (!name.isEmpty() && name != oName) {</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">return name + QLatin1Char('(') + outName + QLatin1Char(')');</pre></div>

<p style="padding: 0; margin: 8px;">}</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Not really, the catch is in the whitespace at the end of <tt style="background: #ebebeb; font-size: 13px;">name</tt> (then <tt style="background: #ebebeb; font-size: 13px;">"Vendor Model " == "Vendor Model"</tt> is never true)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27675">https://phabricator.kde.org/D27675</a></div></div><br /><div><strong>To: </strong>dvratil, Plasma, romangg<br /><strong>Cc: </strong>ahmadsamir, romangg, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>