[gstreamer-bugs] [Bug 595231] [pulsesink] Lowers volume after every new track

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=595231
  GStreamer | gst-plugins-good | 0.10.15

--- Comment #22 from Lennart Poettering <lennart at poettering.net> 2009-10-17 12:35:49 UTC ---
(In reply to comment #19)
> (In reply to comment #17)
> > That series also includes a patch for bug 595029. So if you merge the series
> > make sure to close that bug too.
> > 
> > And the git URL is this one:
> > 
> > git://git.0pointer.de/gst-plugins-good.git
> 
> Hm, shouldn't the volume setting in pa_stream_connect_playback() go away and be
> replaced by pa_context_set_sink_input_volume()? Because former is relative
> volume and latter absolute...

Yes and no. Even worse than setting the volume is setting the volume
asynchronously in two steps. 

The proper fix for this is that we use a new flag I just added to PA which
allows clients to select whether they want to specify relative or absolute
volumes. For now I'd like to leave the current logic where the volume before
and after stream setup can mean something different. (which they can anyway,
since before the stream setup we don't know anything about the device and the
volume capabilities of the device, but afterwards we do)

I'll prep a patch for gst as soon as this new flag is stabilizied.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.