[gstreamer-bugs] [Bug 628996] pulsesink broken after shared context patch (bug #624338)

[GStreamer (bugzilla.gnome.org)]

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.