[pulseaudio-discuss] Routing of audio to loudspeakers or headphones

Colin Guthrie gmane at colin.guthr.ie
Tue Sep 14 07:10:00 PDT 2010


'Twas brillig, and Colin Guthrie at 14/09/10 15:00 did gyre and gimble:
> 'Twas brillig, and Kim Therkelsen at 14/09/10 10:48 did gyre and gimble:
>> Based on the advice from Colin Guthrie I am now trying to modify the
>> module-ladspa-sink module so that it takes a "master_port" argument and
>> only creates the virtual sink when the master sink's port matches the
>> argument value (and unload it when it changes). I think it should be
>> possible to leave the master_port argument out and let the functionality
>> fall back to the original functionality to ensure backward compatibility. 
> 
> Yes, backwards compatibility is a good plan.
> 
>> Furthermore I have updated module-ladspa-sink to support multiple audio
>> channels based on:
>> http://www.pulseaudio.org/attachment/ticket/410/pulseaudio-0.9.12-multichannel-ladspa.patch
> 
> Oh dear, that's been there an awful long time :(
> 
> I guess it's been missed due to general Trac weirdness :(

Incidentally, just on this particular point....

If you do update module-ladspa-sink and bring that patch up to date (I'm
not sure how much work was needed to make it work with current git
master?), then please submit this patch separately to the one adding the
master_port support.

It is *much* better to only commit one change at a time rather than
group up unrelated changes.

Hope that makes sense but if not, just ask.

Col



-- 

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

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mandriva Linux Contributor [http://www.mandriva.com/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]




More information about the pulseaudio-discuss mailing list