ngraham added inline comments.

INLINE COMMENTS

> microphoneindicator.cpp:264
>  {
> -    // If there are no microphones present, there's nothing to record
> -    if (m_sourceModel->rowCount() == 0) {

I don't think this needs to be removed. If an app is "monitoring audio" but 
there 's no audio recording device, that it hardly matters, right?

REPOSITORY
  R115 Plasma Audio Volume Applet

REVISION DETAIL
  https://phabricator.kde.org/D29312

To: kurmikon, #vdg, #plasma, drosca, broulik
Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to