<table><tr><td style="">drosca added a comment.
</td></tr></table><br /><div><div><p>I've made some changes that I wrote about in last comment.</p>

<p>There are still issues that I think needs to be sorted out:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">Profile</tt> doesn't derive from <tt style="background: #ebebeb; font-size: 13px;">PulseObject</tt></li>
<li class="remarkup-list-item">casting d-ptr is weird <tt style="background: #ebebeb; font-size: 13px;">obj->Stream::d->update()</tt> - but this is only in .cpp so I guess it's fine for now</li>
<li class="remarkup-list-item">move update() function to Private class everywhere</li>
<li class="remarkup-list-item">remove all libpulse includes in public headers</li>
<li class="remarkup-list-item">some classes still don't have d-ptr</li>
<li class="remarkup-list-item">rename <tt style="background: #ebebeb; font-size: 13px;">pulseaudio.h</tt> include (probably to <tt style="background: #ebebeb; font-size: 13px;">modules.h</tt>)</li>
<li class="remarkup-list-item">constants from <tt style="background: #ebebeb; font-size: 13px;">Context</tt> (Normal|Minimal|Maximal Volume) should probably go just into <tt style="background: #ebebeb; font-size: 13px;">PulseAudioQt</tt> namespace</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>drosca<br /><strong>Cc: </strong>davidedmundson, michaelh, akrutzler, apol, sitter, drosca, KDE Connect, nicolasfella, yannux, Danial0_0, johnq, ragreen, Pitel, adeen-s, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, ZrenBot, seebauer, ngraham, bruns, bugzy, MayeulC, menasshock, lesliezhai, ali-mohamed, jensreuterberg, ach, abetts, sebas, mart<br /></div>