<table><tr><td style="">drosca added a comment.
</td></tr></table><br /><div><div><p>Alright, would be great if you push only code that builds. I know that it is very very WIP, but still. Also please follow KDE code style (eg. <tt style="background: #ebebeb; font-size: 13px;">Pointer *d</tt>, not <tt style="background: #ebebeb; font-size: 13px;">Pointer* d</tt>) and fix your text editor to strip trailing whitespace (git will also highlight this in diff).</p>

<p>As for the code, it looks fine. One thing to note though, we can't export anything in <tt style="background: #ebebeb; font-size: 13px;">maps.h</tt>, so there needs to be some other API to access these collections. Internally, we still want to use, but it needs special public API - something like <tt style="background: #ebebeb; font-size: 13px;">QMap<quint32, T></tt> with added/removed signals should be enough.</p></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>drosca<br /><strong>Cc: </strong>michaelh, akrutzler, apol, sitter, drosca, KDE Connect, nicolasfella, ragreen, adeen-s, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, ZrenBot, seebauer, ngraham, bugzy, MayeulC, menasshock, lesliezhai, ali-mohamed, jensreuterberg, ach, abetts, sebas, mart<br /></div>