[Bug 719684] videodecoder: Allocation query is always at least sent twice

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Dec 3 04:11:22 PST 2013


https://bugzilla.gnome.org/show_bug.cgi?id=719684
  GStreamer | gst-plugins-base | git

--- Comment #9 from Julien Isorce <julien.isorce at gmail.com> 2013-12-03 12:11:15 UTC ---
It was mainly to avoid double check of the flag, In the previous patch, there
were gst_pad_needs_reconfigure and then gst_pad_check_reconfigure.
It also allows to have more control on what we choose to do inside
GstVideoDecoder so that we do not depends on the gst_video_decoder_negotiate
behavior that can be called from outside.

Then I used this opportunity to also remove the lock even if recursive. Maybe
the function should be renamed to gst_video_decoder_negotiate_unchecked or
something.

Let me know what do you think (and also about comment #7) and I will remake the
whole patch :)

-- 
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.


More information about the gstreamer-bugs mailing list