<table><tr><td style="">sitter added inline comments.
</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/D26935">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/D26935#inline-152355">View Inline</a><span style="color: #4b4d51; font-weight: bold;">context.cpp:600</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: #aa4000">void</span> <span class="n">Context</span><span style="color: #aa2211">::</span><span class="n">loadModule</span><span class="p">(</span><span style="color: #aa4000">const</span> <span class="n">QString</span> <span style="color: #aa2211">&</span><span class="n">name</span><span class="p">,</span> <span style="color: #aa4000">const</span> <span class="n">QString</span> <span style="color: #aa2211">&</span><span class="n">argument</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Should this maybe return a bool? If the operation as a whole fails it seems that should be reflected somehow. Or does this emit an error via event subscription?</p></div></div><br /><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/D26935#inline-152354">View Inline</a><span style="color: #4b4d51; font-weight: bold;">context.cpp:607</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: #aa4000">const</span> <span class="n">QByteArray</span> <span class="n">argumentData</span> <span style="color: #aa2211">=</span> <span class="n">argument</span><span class="p">.</span><span class="n">toUtf8</span><span class="p">();</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">if</span> <span class="p">(</span><span style="color: #aa2211">!</span><span class="n">PAOperation</span><span class="p">(</span><span class="n">pa_context_load_module</span><span class="p">(</span><span class="n">d</span><span style="color: #aa2211">-></span><span class="n">m_context</span><span class="p">,</span> <span class="n">nameData</span><span class="p">.</span><span class="n">constData</span><span class="p">(),</span> <span class="n">argumentData</span><span class="p">.</span><span class="n">constData</span><span class="p">(),</span> <span style="color: #aa4000">nullptr</span><span class="p">,</span> <span style="color: #aa4000">nullptr</span><span class="p">)))</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">qCWarning</span><span class="p">(</span><span class="n">PULSEAUDIOQT</span><span class="p">)</span> <span style="color: #aa2211"><<</span> <span style="color: #766510">"pa_context_load_module failed"</span><span class="p">;</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;"><a href="https://doc.qt.io/qt-5/qtglobal.html#qUtf8Printable" class="remarkup-link" target="_blank" rel="noreferrer">https://doc.qt.io/qt-5/qtglobal.html#qUtf8Printable</a></p></div></div><br /><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/D26935#inline-152357">View Inline</a><span style="color: #4b4d51; font-weight: bold;">context.cpp:612</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: #aa4000">void</span> <span class="n">Context</span><span style="color: #aa2211">::</span><span class="n">unloadModule</span><span class="p">(</span><span style="color: #aa4000">const</span> <span class="n">quint32</span> <span class="n">index</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">As I've mentioned in private message, I'd do away with this. The only way you, as client, get to the index is through the Module object, so you may as well pass in the Module*, thus rendering the index variant moot from an object POV.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R994 Pulseaudio Qt Bindings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26935">https://phabricator.kde.org/D26935</a></div></div><br /><div><strong>To: </strong>bshah, sitter, nicolasfella<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>