[Bug 755510] vpx: create base class for vpx encoders and decoders

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |slomo at coaxion.net

--- Comment #4 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
(In reply to Nicolas Dufresne (stormer) from comment #3)

> ::: ext/vpx/gstvp8dec.c
> @@ +129,3 @@
>  {
> +  gst_tag_list_add (list, GST_TAG_MERGE_REPLACE,
> +      GST_TAG_VIDEO_CODEC, "VP8 video", NULL);
> 
> Could also be a static const char on the class structure.

Note that there is also API for that in pbutils to set the codec tag based on
the caps. Then we only need to hardcode this string in a single place.

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