[pulseaudio-discuss] [PATCH] combine: Handle reappearing slave sinks in non-automatic mode.

Colin Guthrie gmane at colin.guthr.ie
Fri Nov 12 01:15:56 PST 2010


'Twas brillig, and Antti-Ville Jansson at 12/11/10 07:52 did gyre and
gimble:
> Earlier, if slave sinks were unlinked in non-automatic mode, their
> re-appearance was disregarded. Now they are added back to the list of outputs.
> 
> Signed-off-by: Antti-Ville Jansson <antti-ville.jansson at digia.com>
> Reviewed-by: Tanu Kaskinen <tanu.kaskinen at digia.com>

Cool, thanks. I'll review/merge over the weekend (not much reviewing
needed seeing as tanuk has already reviewed tho' - and it's quite short!)

I would personally go one step further and allow for invalid slave names
to be supplied at startup the logic being:

 * If no slaves exist, just idle and wait for the necessary hooks before
loading the actual combine sink.
 * If one of our "watched" slaves turns up, and we have not yet created
our combine sink, then create it.
 * If all our slaves disappear, unload our sink but keep the module
loaded and go back to idling and waiting for sinks to appear again.

I think the above changes would make the module much more useful. It may
take a little more engineering however because of the slight restructure.

If anyone fancies taking a crack at that, I'd be more than happy to
offer advice :D

Cheers

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mageia Contributor [http://www.mageia.org/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]




More information about the pulseaudio-discuss mailing list