[Bug 784599] kmssink: support videooverlay interface

[GStreamer (GNOME Bugzilla)]

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

--- Comment #41 from Haihua Hu <jared.hu at nxp.com> ---
(In reply to Víctor Manuel Jáquez Leal from comment #39)
> (In reply to Haihua Hu from comment #38)
> > Review of attachment 356483 [details] [review] [review]:
> > 
> > ::: sys/kms/gstkmssink.c
> > @@ +97,3 @@
> > +  GstKMSSink *self = GST_KMS_SINK (overlay);
> > +
> > +  if (width == 0 || height == 0)
> > 
> > We need add FIXME here if it should be fix later， and I think below is better
> 
> There is no reason to merge an incomplete feature. It doesn't seem too
> complex to get it done. Can you update the patch?

Sorry for late, I have update the patch.

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