[gstreamer-bugs] [Bug 638844] [PATCH] Fix pad callbacks so they handle when parent goes away

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=638844
  GStreamer | gstreamer (core) | git

--- Comment #5 from Ole André Vadla Ravnås <ole.andre.ravnas at tandberg.com> 2011-01-10 13:15:50 UTC ---
(In reply to comment #4)
> I've come accross the same problem and proposed the same solution. That said, I
> think the best solution is to change GstPad to do that for us.
> 
> *** This bug has been marked as a duplicate of bug 631580 ***

Missed out on that bug -- thanks! The only immediate downside I can see with
changing GstPad is that it would potentially result in a few extra ref/unrefs
for the (few) callbacks that don't need the parent. But on the upside it would
make it possible to combine the two consecutive
GST_PAD_LOCK()/GST_PAD_UNLOCK()s into one. So I'm leaning towards changing
GstPad to just do this for us.

Thoughts, anyone?

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