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

[Ashok Kumar Mishra]

On Wed, Nov 7, 2018 at 10:07 PM Vittorio Giovara <vittorio.giovara at gmail.com>
wrote:

> Please try the attached file.
>
> Thanks,
> Vittorio
>
> On Wed, Nov 7, 2018 at 2:24 AM Ashok Kumar Mishra <
> ashok at multicorewareinc.com> wrote:
>
>> 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
>>>
>> _______________________________________________
>> x265-devel mailing list
>> x265-devel at videolan.org
>> https://mailman.videolan.org/listinfo/x265-devel
>>
>
>
> --
> Vittorio
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel


Pushed.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20181112/538340d1/attachment.html>