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

[Ashok Kumar Mishra]

Hi Vittorio,

Unable to apply this patch on the latest tip, Can you please send the patch
once again so that we can push this into default asap.

Thanks.

On Wed, Nov 7, 2018 at 12:01 PM Praveen Tiwari <praveen at multicorewareinc.com>
wrote:

> 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
>>
> _______________________________________________
> 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/96299170/attachment.html>