<table><tr><td style="">graesslin requested changes to this revision.<br />graesslin added inline comments.<br />This revision now requires changes to proceed.
</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/D10040" rel="noreferrer">View Revision</a></tr></table><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/D10040#inline-46305" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">outputdevice.xml:104-107</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span style="color: #00702a"><arg</span> <span style="color: #354bb3">name=</span><span style="color: #766510">"serialNumber"</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"string"</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">             <span style="color: #354bb3">summary=</span><span style="color: #766510">"textual representation of serial number"</span><span style="color: #00702a">/></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span style="color: #00702a"><arg</span> <span style="color: #354bb3">name=</span><span style="color: #766510">"eisaId"</span> <span style="color: #354bb3">type=</span><span style="color: #766510">"string"</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">             <span style="color: #354bb3">summary=</span><span style="color: #766510">"textual representation of EISA identifier"</span><span style="color: #00702a">/></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I'm not sure whether it's allowed to add arguments to an existing event. This would result in incompatibilities. You can make the server only emit to clients having the version, but then you actually broke any client which only has version 1.</p>

<p style="padding: 0; margin: 8px;">To be really compatible you need to add a new event.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10040" rel="noreferrer">https://phabricator.kde.org/D10040</a></div></div><br /><div><strong>To: </strong>dvratil, graesslin, sebas<br /><strong>Cc: </strong>davidedmundson, plasma-devel, Frameworks, schernikov, ZrenBot, ngraham, alexeymin, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein<br /></div>