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

[Vittorio Giovara]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20181107/538e4979/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: encoder.patch
Type: application/octet-stream
Size: 1255 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20181107/538e4979/attachment.obj>