[Bug 796106] video: Support alternate interlaced video

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=796106

Nicolas Dufresne (ndufresne) <nicolas at ndufresne.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #373401|none                        |needs-work
             status|                            |

--- Comment #32 from Nicolas Dufresne (ndufresne) <nicolas at ndufresne.ca> ---
Review of attachment 373401:
 --> (https://bugzilla.gnome.org/review?bug=796106&attachment=373401)

::: gst-libs/gst/video/video-info.c
@@ +698,3 @@
+    height = (gsize) info->height / 2;
+  else
+    height = (gsize) info->height;

Maybe move up the patch adding GST_VIDEO_INFO_FIELD_HEIGHT() and use it here ?

::: gst-libs/gst/video/video-info.h
@@ +34,3 @@
+ * GST_CAPS_FEATURE_FORMAT_INTERLACED:
+ *
+ * Name of the caps feature for indicating the stream is interlaced. Currently

What about "Name of the caps feature indicating that the stream is interlaced."
?

@@ +35,3 @@
+ *
+ * Name of the caps feature for indicating the stream is interlaced. Currently
+ * it is only used for video.

Since: 1.16

@@ +56,3 @@
+ *     @GST_VIDEO_BUFFER_FLAG_TF or @GST_VIDEO_BUFFER_FLAG_BF indicates if
+ *     the buffer is carrying the top or bottom field, respectively. The top
and
+ *     bottom buffers are expected to alternate in the pipeline, with this
mode.

(Since 1.16)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.