On 2013-06-29 15:27, Nicolas BERTRAND wrote:
Le 28/06/13 13:29, Diego Biurrun a écrit :
On Fri, Jun 28, 2013 at 12:24:56PM +0200, Nicolas Bertrand wrote:
--- a/libavcodec/jpeg2kdec.c
+++ b/libavcodec/jpeg2kdec.c
@@ -98,26 +95,22 @@ typedef struct Jpeg2KDecoderContext {
s->bit_index--;
- 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.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
