commit 4a4e5b43e50e62c4d420aaf9af7ad68295117508
author Wan-Teh Chang <wtc@google.com> Mon Nov 04 09:33:07 2024 -0800
committer Wan-Teh Chang <wtc@google.com> Mon Nov 04 10:53:08 2024 -0800
tree d8337ce86b11c9854b996a920cc7f2edaf874f87
parent dab2aa5ef42ea20ba0c98be88cf655a34cf3512b

Test bitCount == 0 in avifROStreamSkipBits()

Testing bitCount == 0 is easier to understand and avoids unnecessary
work if stream->numUsedBitsInPartialByte == 0 and bitCount == 0.

Suggested by Yannis Guyon.

Bug: b:376733153

src/stream.c

diff --git a/src/stream.c b/src/stream.c
index 12a1eff..b9b8488 100644
--- a/src/stream.c
+++ b/src/stream.c
@@ -153,7 +153,7 @@
         const size_t padding = AVIF_MIN(8 - stream->numUsedBitsInPartialByte, bitCount);
         stream->numUsedBitsInPartialByte = (stream->numUsedBitsInPartialByte + padding) % 8;
         bitCount -= padding;
-        if (stream->numUsedBitsInPartialByte != 0) {
+        if (bitCount == 0) {
             return AVIF_TRUE;
         }
     }