[Bug 784599] kmssink: support videooverlay interface

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Thu Jul 6 13:50:47 UTC 2017


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

--- Comment #7 from Víctor Manuel Jáquez Leal <vjaquez at igalia.com> ---
Review of attachment 355004:
 --> (https://bugzilla.gnome.org/review?bug=784599&attachment=355004)

::: sys/kms/gstkmssink.c
@@ +1333,3 @@
+    self->original_heigth = GST_VIDEO_INFO_HEIGHT (&self->vinfo);
+  }
+

wouldn't be clearer to set this in gst_kms_sink_set_caps() when reconfigure is
TRUE?

::: sys/kms/gstkmssink.h
@@ +77,3 @@
+  gint preferred_y;
+  gint preferred_width;
+  gint preferred_height;

wouldn't be better to use a GstVideoRectangle here instead of these four
integers?

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