D29312: Change microphoneindicator for reporting audio monitors

Giusy Margarita noreply at phabricator.kde.org
Thu Apr 30 20:57:36 BST 2020


kurmikon added a comment.


  In D29312#660704 <https://phabricator.kde.org/D29312#660704>, @nicolasfella wrote:
  
  > In D29312#660702 <https://phabricator.kde.org/D29312#660702>, @kurmikon wrote:
  >
  > > In D29312#660670 <https://phabricator.kde.org/D29312#660670>, @nicolasfella wrote:
  > >
  > > > > but due to a lack in qt libraries
  > > >
  > > > Can you elaborate on this? What is Qt lacking?
  > >
  > >
  > > I'm not an expert, so I don't really know. Reading the bug report, there's no way to discern input devices from monitor sinks. So if you want to report applications that are using a microphone, you end up showing applications like PuseEffects that create a monitor sink. Those applications can use a microphone but in most cases don't, because PulseEffects is mostly used to apply effects to output streams (but need to record those streams effectively).
  >
  >
  > Qt is not really involved in this. What matters is what libpulse offers, and it seems to me that we can check whether a sink is a monitor (https://freedesktop.org/software/pulseaudio/doxygen/structpa__source__info.html#a5e304b796ce71c7fa54e5a88f3333630). One would need to add a method isMonitor to Sink that reads this information and then we can not show the indicator when it's a monitor
  
  
  I'm afraid that application that really use a microphone, create a monitor sink.

REPOSITORY
  R115 Plasma Audio Volume Applet

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

To: kurmikon, #vdg, #plasma, drosca, broulik
Cc: nicolasfella, 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200430/c4e3f819/attachment.html>


More information about the Plasma-devel mailing list