After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 567748 - http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/ext/pulse/pulsesink.c?r1=1.12&r2=1.13 is bogus
http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/ext/pulse/pulsesink....
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-01-14 13:57 UTC by Lennart Poettering
Modified: 2009-02-17 16:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Lennart Poettering 2009-01-14 13:57:23 UTC 
Please describe the problem:
I am pretty sure that this commit is bogus:

http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/ext/pulse/pulsesink.c?r1=1.12&r2=1.13

The newly introduced mutex is taken only when the threaded mainloop lock is taen as well. As such it is a "submutex" for another mutex and hence redundant.

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Edward Hervey 2009-02-17 16:29:04 UTC 
Is this still valid ?
Comment 2 Jan Schmidt 2009-02-17 16:37:44 UTC 
Nope, ta.