On Fri, Mar 23, 2018 at 5:10 PM, Vittorio Giovara < [email protected]> wrote:
> > > On Fri, Mar 23, 2018 at 6:23 AM, <[email protected]> wrote: > >> # HG changeset patch >> # User Ashok Kumar Mishra <[email protected]> >> # 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 > [email protected] > https://mailman.videolan.org/listinfo/x265-devel > > Agreed.
_______________________________________________ x265-devel mailing list [email protected] https://mailman.videolan.org/listinfo/x265-devel
