[x265] [PATCH] encoder: Do not include CLL SEI message if empty

[Praveen Tiwari]

Hello Vittorio,

Sorry for the late reply,  all of us were on leave due to the Diwali
festival in India.

Thanks for the patch, will run some basic test and push the patch.

Regards,
Praveen


On Wed, Nov 7, 2018 at 12:35 AM Vittorio Giovara <vittorio.giovara at gmail.com>
wrote:

>
>
> On Thu, Nov 1, 2018 at 5:34 PM Vittorio Giovara <
> vittorio.giovara at gmail.com> wrote:
>
>> Some devices render out-of-luminance pixels incorrectly otherwise.
>>
>> ---
>>  source/encoder/encoder.cpp | 11 +++++++----
>>  1 file changed, 7 insertions(+), 4 deletions(-)
>>
>> diff -r fd517ae68f93 source/encoder/encoder.cpp
>> --- a/source/encoder/encoder.cpp    Tue Sep 25 16:02:31 2018 +0530
>> +++ b/source/encoder/encoder.cpp    Thu Nov 01 17:27:51 2018 -0400
>> @@ -2381,10 +2381,13 @@
>>
>>      if (m_param->bEmitHDRSEI)
>>      {
>> -        SEIContentLightLevel cllsei;
>> -        cllsei.max_content_light_level = m_param->maxCLL;
>> -        cllsei.max_pic_average_light_level = m_param->maxFALL;
>> -        cllsei.writeSEImessages(bs, m_sps, NAL_UNIT_PREFIX_SEI, list,
>> m_param->bSingleSeiNal);
>> +        if (m_emitCLLSEI)
>> +        {
>> +            SEIContentLightLevel cllsei;
>> +            cllsei.max_content_light_level = m_param->maxCLL;
>> +            cllsei.max_pic_average_light_level = m_param->maxFALL;
>> +            cllsei.writeSEImessages(bs, m_sps, NAL_UNIT_PREFIX_SEI,
>> list, m_param->bSingleSeiNal);
>> +        }
>>
>>          if (m_param->masteringDisplayColorVolume)
>>          {
>> --
>> Vittorio
>>
>
> ping
> --
> Vittorio
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20181107/a0853d88/attachment.html>