On Fri, May 5, 2017 at 10:32 PM, Luca Barbato <[email protected]> wrote:
> On 5/5/17 8:56 PM, James Almer wrote:
>> On 4/30/2017 12:35 PM, James Almer wrote:
>>> Based on the H264 SEI implementation.
>>>
>>> This will be mainly useful once support for SEI messages that can be
>>> used by the hevc parser are implemented, like Picture Timing.
>>>
>>> Signed-off-by: James Almer <[email protected]>
>>> ---
>>>  libavcodec/hevc_sei.c | 77 
>>> +++++++++++++++++++++++++--------------------------
>>>  libavcodec/hevcdec.c  | 49 +++++++++++++++++---------------
>>>  libavcodec/hevcdec.h  | 44 ++++++++++++++++++-----------
>>>  3 files changed, 92 insertions(+), 78 deletions(-)
>>
>> Ping for set.
>
> HEVCSEIContext doesn't seem a context though, probably HEVCSEIInfo might
> be a better name.
>
> Beside that seems fine to me.
>

The name is consistent with the H264 decoder struct naming, which is
called H264SEIContext, so imho its fine.

- Hendrik
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to