On 06/29/2013 03:27 PM, 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 '&'
In that case =)
http://en.wikipedia.org/wiki/Operators_in_C_and_C%2B%2B#Operator_precedence
Yes, looks more fuzzy to me as well.
lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
