[x265] [PATCH] SEI: fix build error

[Ashok Kumar Mishra]

On Fri, Mar 23, 2018 at 5:10 PM, Vittorio Giovara <
vittorio.giovara at gmail.com> wrote:

>
>
> On Fri, Mar 23, 2018 at 6:23 AM, <ashok at multicorewareinc.com> wrote:
>
>> # HG changeset patch
>> # User Ashok Kumar Mishra <ashok at multicorewareinc.com>
>> # Date 1521800501 -19800
>> #      Fri Mar 23 15:51:41 2018 +0530
>> # Node ID 0634c5a14899b2ed59ce133a35a0ff644887be85
>> # Parent  5be53f09eee6ab28bd9dbb2c0ac447f50eb29655
>> SEI: fix build error
>>
>> diff -r 5be53f09eee6 -r 0634c5a14899 source/encoder/sei.cpp
>> --- a/source/encoder/sei.cpp    Wed Mar 14 17:24:34 2018 +0530
>> +++ b/source/encoder/sei.cpp    Fri Mar 23 15:51:41 2018 +0530
>> @@ -36,12 +36,11 @@
>>  /* count the size of the payload and return the size in bits */
>>  int SEI::countPayloadSize(const SPS& sps)
>>  {
>> -    BitCounter counter;
>> -    int count = 0;
>> +    BitCounter counter;
>>      m_bitIf = &counter;
>>      writeSEI(sps);
>> -    X265_CHECK(0 == (count.getNumberOfWrittenBits() & 7), "payload
>> unaligned\n");
>> -    count = counter.getNumberOfWrittenBits() >> 3;
>> +    X265_CHECK(0 == (counter.getNumberOfWrittenBits() & 7), "payload
>> unaligned\n");
>> +    int count = counter.getNumberOfWrittenBits() >> 3;
>>      return count;
>>  }
>>
>
> nit: you can return the result without the extra variable
> --
> Vittorio
>
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel
>
>

Agreed.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20180323/9fdf75b0/attachment.html>