Hi,

2011/12/15 Måns Rullgård <[email protected]>

> "Ronald S. Bultje" <[email protected]> writes:
> > @@ -311,7 +349,10 @@ static inline unsigned int get_bits1(GetBitContext
> *s){
> >      result <<= index & 7;
> >      result >>= 8 - 1;
> >  #endif
> > -    index++;
> > +#if !UNCHECKED_BITSTREAM_READER
> > +    if (s->index < s->size_in_bits_plus1)
> > +#endif
> > +        index++;
> >      s->index = index;
>
> Why #if rather than if (UNCHECKED_BITSTREAM_READER || ...) here?
>

That way I can put the s->size_in_bits_plus1 declaration in the struct
under the #if, so as to safe 4 bytes of unused data in the struct for
people preferring the unsafe variant.

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

Reply via email to