[Bug 735085] y4mencode : port y4m encoder to use GstVideoEncoder base class

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=735085
  GStreamer | gst-plugins-good | git

Thiago Sousa Santos <thiagossantos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #284476|none                        |reviewed
             status|                            |

--- Comment #3 from Thiago Sousa Santos <thiagossantos at gmail.com> 2014-09-03 23:50:21 UTC ---
Review of attachment 284476:
 --> (https://bugzilla.gnome.org/review?bug=735085&attachment=284476)

Thanks for the update, seems to only have one minor issue before it can be
merged.

::: gst/y4m/gsty4mencode.c
@@ +157,3 @@
+  output_state =
+      gst_video_encoder_set_output_state (encoder,
+      gst_caps_new_empty_simple ("video/x-raw"), state);

This seems wrong, the output caps isn't video/x-raw but
"application/x-yuv4mpeg, y4mversion=2" according to the template caps.

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