[gst-devel] [gst-cvs] dolphy gst-plugins-base: gst-plugins-base/ gst-plugins-base/ext/alsa/

[Tim Müller]

On Mon, 2008-02-11 at 20:04 +0000, Jan Schmidt wrote:

> dolphy at kemper.freedesktop.org wrote:
> > CVS Root:       /cvs/gstreamer
> > Module:         gst-plugins-base
> > Changes by:     dolphy
> > Date:           Mon Feb 11 2008  17:03:32 UTC
> >
> > Log message:
> > 2008-02-11  Julien Moutte  <julien at fluendo.com>
> >         Patch by: Alan Peevers <peeves at pacbell.net>
> >         * ext/alsa/gstalsasink.c: (gst_alsasink_delay): Take appropriate
> >         lock when calling alsa methods.
> >   
> Is there a bug report or a test case where this fixes a known issue?
> 
> If not, it's better not to take an extra lock in the clock get_time path 
> if we can avoid it.

I've reverted this for now, since it breaks playback for me, and a short
discussion on IRC indicated that the patch in its current form might be
a bad idea.

 Cheers
  -Tim