On 10/05/18 06:47, Haihao Xiang wrote:
> Similar to H264, cbs_h265_{read, write}_nal_unit() can handle HEVC
> prefix SEI NAL units. Currently mastering display colour volume SEI
> message is added only, we may add more SEI message if needed later
> 
> v2: Fix coding style and rebase the code
> 
> v3: Fix assertion failure when running the command below:
> ffmpeg -i input-with-sei.mkv -c:v copy -bsf:v trace_headers -f null -
> Fix the ranges for max_display_mastering_luminance and 
> min_display_mastering_luminance
> 
> Signed-off-by: Haihao Xiang <haihao.xi...@intel.com>
> ---
>  libavcodec/cbs_h2645.c                |  45 ++++++++++
>  libavcodec/cbs_h265.h                 |  36 ++++++++
>  libavcodec/cbs_h265_syntax_template.c | 158 
> ++++++++++++++++++++++++++++++++++
>  3 files changed, 239 insertions(+)
> 
> ...
>  
> +typedef struct H265RawSEIMasteringDiplayColourVolume {

I fixed this typo                     ^^ which appears in several places (a bad 
feature of code completion, I guess :P ).

Then tested, all looks good, applied.

Thanks!

- Mark
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to