[pulseaudio-discuss] [PATCH] native: Don't save device, volume or mute of new streams.

Colin Guthrie gmane at colin.guthr.ie
Thu Apr 5 08:13:57 PDT 2012


'Twas brillig, and Tanu Kaskinen at 05/04/12 14:24 did gyre and gimble:
> Specifying the volume when creating a new stream is not an
> equivalent act as setting the volume with a volume control
> application. When creating a new stream, stream-restore
> shouldn't save the volume, but when changing the volume,
> then saving it is ok. For example, when I say
> "paplay --volume=10000 somefile.wav", I mean that I want the
> new stream to have volume 10000. I don't mean that also
> future paplay invocations (without the --volume option)
> should have that same volume.

My initial reaction on seeing this commit was, I would consider passing
a volume with a new stream as user input...

> This patch effectively reverts
> 546bcf3f2f9711f0d08c21c3b775994844e7e2a2.

... which is exactly what Lennart said in this commit!  Spooky!

But on reflection, I'm not sure what I really feel about this.

All in all, I think we want to discourage clients from setting volumes
like this anyway (paplay is arguably a special case), as they are trying
to outsmart us and override user settings (e.g. how does the app know if
it is controlling a category wide volume or just itself, or the volume
as was on the Headphones port vs. the Speaker - so it shouldn't even try!)

But yeah, I don't see any massive problem here regarding not considering
it user input, but wouldn't be surprised if a counter argument cropped up...

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