<div>broulik added a comment.</div><br /><div><div><p>Nice</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div>src/maps.h:92 Not too fond of allocating a list just to get an item from it, can we perhaps do something like (m_data.constBegin() + index).value();</div><div>src/pulseaudio.h:48 You don't need slots to connect to with the new connect syntax iirc</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAPA Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1393" rel="noreferrer">https://phabricator.kde.org/D1393</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>drosca, Plasma<br /><strong>Cc: </strong>broulik, plasma-devel, sebas<br /></div>