[Bug 607742] add gst_event_new_force_key_unit: gst-plugins-base/gst-libs/gst/video/video.c

[GStreamer (bugzilla.gnome.org)]

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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #13 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2011-03-29 22:37:58 UTC ---
> While we're at it, maybe we should add some params to say ... (snip)
> We may also want a way to specify ... (snip)

In order to move this forward: what is the essential stuff that such an event
should *always* have?

Reading
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/tree/docs/design/draft-keyframe-force.txt
it sounds like the upstream event just has a name, while the downstream event
should at least carry timing info and the all-headers flag.

If this is right, I guess we could do something like this (using CUSTOM_BOTH
then).

  GstEvent * gst_event_new_force_key_unit (void);

  void          
gst_event_set_force_key_unit_{running_time,time_stamp,whatever} (event, ...);

We can then add more API for other "nice to have" things later.

Thoughts?

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