[Bug 721666] videodecoder: push newsegment earlier for reverse playback

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #16 from Aleix Conchillo Flaqué <aleix at oblong.com> 2014-01-11 01:16:08 UTC ---
Review of attachment 265913:
 --> (https://bugzilla.gnome.org/review?bug=721666&attachment=265913)

::: gst-libs/gst/video/gstvideodecoder.c
@@ +1968,3 @@
+        dec->priv->pending_events =
+            g_list_append (dec->priv->pending_events, event);
+        frame->events = g_list_delete_link (frame->events, cur);

Trying to understand the patch.

Why do we need to move the event from the frame and put it in the pending
events list?

Don't they get all pushed in gst_video_decoder_prepare_finish_frame? First, the
pending events and then the frame events?

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