[Bug 796519] Add AV1 codec parser

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Jul 11 07:50:04 UTC 2018


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

--- Comment #19 from Georg Ottinger <g.ottinger at gmx.at> ---

> I have a concern about this. The temproal_id, frame_header may only be
> related to one frame, why we put this in parser? 
> I suggest we follow some logic like this
> 1. If some information only applied in on a frame we can put it to
> FrameHeader.
> 2. If some information applied to multi frame
>        if caller need knows this information, we can put it to the public.
>        if the caller does not need to know this information, we can put it
> to a private c file
> thanks

this explaination was very helpful. I should have guessed it before that
private means private (like in a object orientate way)

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