[x265] [PATCH] Fix payload size for SEIRecoveryPoint
Aarthi Priya Thirumalai
aarthi at multicorewareinc.com
Fri Mar 16 16:19:32 CET 2018
Yeah but I didn't add it to that flag as this is not a hrd related
parameter. But I will clean up this function and resend the patch.
pls ignore this
Aarthi Priya Thirumalai
Technical Lead, UHDkit
Multicoreware Inc
On Fri, Mar 16, 2018 at 5:35 PM, Ashok Kumar Mishra <
ashok at multicorewareinc.com> wrote:
>
>
> On Fri, Mar 16, 2018 at 4:24 PM, <aarthi at multicorewareinc.com> wrote:
>
>> # HG changeset patch
>> # User Aarthi Thirumalai <aarthi at multicorewareinc.com>
>> # Date 1521028474 -19800
>> # Wed Mar 14 17:24:34 2018 +0530
>> # Node ID 265d516af978a209bd22a6ae43487cecd01c4503
>> # Parent d7c26df32fae052b7e895fee9bda1c22b24cc44b
>> Fix payload size for SEIRecoveryPoint
>>
>> diff -r d7c26df32fae -r 265d516af978 source/encoder/sei.cpp
>> --- a/source/encoder/sei.cpp Tue Mar 13 13:40:13 2018 +0530
>> +++ b/source/encoder/sei.cpp Wed Mar 14 17:24:34 2018 +0530
>> @@ -41,8 +41,9 @@
>> uint32_t type = m_payloadType;
>> m_bitIf = &bs;
>> BitCounter count;
>> + uint32_t payloadSize = m_payloadSize;
>> bool hrdTypes = (m_payloadType == ACTIVE_PARAMETER_SETS ||
>> m_payloadType == PICTURE_TIMING || m_payloadType == BUFFERING_PERIOD);
>>
>
> I think it is not required to check the condition "m_payloadType ==
> RECOVERY_POINT" in other places
> if you add this in above flag hrdTypes.
>
>
>> - if (hrdTypes)
>> + if (hrdTypes || m_payloadType == RECOVERY_POINT)
>> {
>> m_bitIf = &count;
>> /* virtual writeSEI method, write to bit counter to determine
>> size */
>> @@ -53,10 +54,10 @@
>> WRITE_CODE(0xff, 8, "payload_type");
>> }
>> WRITE_CODE(type, 8, "payload_type");
>> - uint32_t payloadSize;
>> - if (hrdTypes || m_payloadType == USER_DATA_UNREGISTERED ||
>> m_payloadType == USER_DATA_REGISTERED_ITU_T_T35)
>> + if (hrdTypes || m_payloadType == USER_DATA_UNREGISTERED ||
>> m_payloadType == USER_DATA_REGISTERED_ITU_T_T35 ||
>> + m_payloadType == RECOVERY_POINT)
>> {
>> - if (hrdTypes)
>> + if (hrdTypes || m_payloadType == RECOVERY_POINT)
>> {
>> X265_CHECK(0 == (count.getNumberOfWrittenBits() & 7),
>> "payload unaligned\n");
>> payloadSize = count.getNumberOfWrittenBits() >> 3;
>> @@ -65,13 +66,10 @@
>> payloadSize = m_payloadSize + 16;
>> else
>> payloadSize = m_payloadSize;
>> -
>> - for (; payloadSize >= 0xff; payloadSize -= 0xff)
>> - WRITE_CODE(0xff, 8, "payload_size");
>> - WRITE_CODE(payloadSize, 8, "payload_size");
>> }
>> - else
>> - WRITE_CODE(m_payloadSize, 8, "payload_size");
>> + for (; payloadSize >= 0xff; payloadSize -= 0xff)
>> + WRITE_CODE(0xff, 8, "payload_size");
>> + WRITE_CODE(payloadSize, 8, "payload_size");
>> /* virtual writeSEI method, write to bs */
>> writeSEI(sps);
>> }
>>
>> _______________________________________________
>> 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/20180316/291ea325/attachment.html>
More information about the x265-devel
mailing list