-        res |= (*s->buf >> s->bit_index) & 1;
+        res |= (bytestream2_peek_byte(&s->g) >> s->bit_index) & 1;
pointless ()
Sorry but the () not seems pointless to me :

res |= bytestream2_peek_byte(&s->g) >> s->bit_index & 1;

loks more fuzzy, I m not sure which have the priority between '>>' and
'&'

The expression has left to right precedence, as you read it.
when I read code, I find boring to think about precedence. the () are useful int hat case

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to