<table><tr><td style="">sitter added a comment.
</td></tr></table><br /><div><div><p>Random complaints I stumbled upon:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">m_context = pa_context_new(api, "QPulse");</tt> this should decidedly not hardcode <tt style="background: #ebebeb; font-size: 13px;">QPulse</tt> but use <a href="https://doc.qt.io/qtthe-5/qcoreapplication.html#applicationName-prop" class="remarkup-link" target="_blank" rel="noreferrer">https://doc.qt.io/qtthe-5/qcoreapplication.html#applicationName-prop</a> (or maybe even displayName iff qapp can be cast to qguiapplication)</li>
<li class="remarkup-list-item">along the same notion, Context should probably have a function <tt style="background: #ebebeb; font-size: 13px;">setName</tt> for <tt style="background: #ebebeb; font-size: 13px;">pa_context_set_name</tt> in case the qapp name isn't the desired name (e.g. with a plasmoid the qapp is plasma, but the app name should probably be the individual plasmoid)</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, sitter<br /><strong>Cc: </strong>davidedmundson, michaelh, akrutzler, apol, sitter, drosca, KDE Connect, nicolasfella, jdvr, 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>