<table><tr><td style="">drosca added a comment.
</td></tr></table><br /><div><div><p>Looks fine, except some issues with methods that should be hidden:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Device::updateDevice</tt> should be in private class</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Port::setInfo</tt> should be in private class</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Profile::setInfo</tt> should be in private class</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">PulseObject::updatePulseObject</tt> should be in private class</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Stream::updateStream</tt> should be in private class</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Server::reset</tt> should not be public</li>
</ul></div></div><br /><div><strong>TASK DETAIL</strong><div><a href="https://phabricator.kde.org/T7994">https://phabricator.kde.org/T7994</a></div></div><br /><div><strong>To: </strong>nicolasfella, drosca<br /><strong>Cc: </strong>davidedmundson, michaelh, akrutzler, apol, sitter, drosca, KDE Connect, nicolasfella, yannux, Danial0_0, johnq, ragreen, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, ZrenBot, seebauer, ngraham, bruns, bugzy, MayeulC, menasshock, lesliezhai, ali-mohamed, jensreuterberg, ach, abetts, sebas, mart<br /></div>