[Bug 757597] Codecparsers: add vp9 parser

[GStreamer (GNOME Bugzilla)]

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

--- Comment #5 from sreerenj <bsreerenj at gmail.com> ---
(In reply to Sebastian Dröge (slomo) from comment #4)
> So is there some private stuff to hide in the VP8 parser too? :)

Not really :)
In gstreamerr-vaapi perspective the implementation is fine..But if you consider
it from a codec-analyzer view point there are issues. For eg: if some fields
are not available for a particular frame, they are copying from previous frame.
That is enough for gst-vaapi or any other similar use cases. But if you really
wanna print the values of fields only if exists in a frame(thats what analyzer
is needed) then it won't work.

BTW, vp9 has more tracking stuffs(eg: sizes of up to 8 reference frames)and I
don't wanna expose those to public stuct.

> Note that you'll have some padding to the parser struct if you want to be
> able to extend it later because of the way how you implemented the private
> stuff. A better solution would be to allocate the private data *before* the
> public data, and just have a pointer back to it.

Okay, It was like that initially. Then I thought itz better to avoid exposing
even the pointer in public struct :)

> Or just allocate it
> independently.

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