[gstreamer-bugs] [Bug 628996] pulsesink broken after shared context patch (bug #624338)
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Mon Sep 13 07:01:02 PDT 2010
https://bugzilla.gnome.org/show_bug.cgi?id=628996
GStreamer | gst-plugins-good | git
Stefan Kost (gstreamer, gtkdoc dev) <ensonic> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Target Milestone|HEAD |0.10.26
--- Comment #17 from Stefan Kost (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> 2010-09-13 14:00:59 UTC ---
commit f62dc6976b611384c98efb37d407b5299daf8c17
Author: Stefan Kost <ensonic at users.sf.net>
Date: Mon Sep 13 16:24:26 2010 +0300
pulsesink: rework context sharing
We also need to share the main-loop threads as this owns the context. Thus
have
a class wide main-loop thread. From this we create a context per
client-name.
Instead of always looking up the context, we keep this with the instance.
The
reverse mapping is only needed in pulse singal handlers. This saves a lot
of
locking. Also one signal handler becomes simpler as ther eis only one
mainloop
to notify.
Now valgind happy - no leaks, no bad reads/writes.
This reverts major parts of commit
69a397c32f4baf07a7b2937c610f9e8f383e9ae9.
Fixes #628996
--
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
More information about the Gstreamer-bugs
mailing list