On Thu, Dec 29, 2011 at 05:32:09PM +0100, Diego Biurrun wrote:
> On Thu, Dec 29, 2011 at 05:26:50PM +0100, Kostya Shishkov wrote:
> > ---
> > libavcodec/indeo4.c | 10 ++++++----
> > 1 files changed, 6 insertions(+), 4 deletions(-)
>
> OK
>
> > --- a/libavcodec/indeo4.c
> > +++ b/libavcodec/indeo4.c
> > @@ -337,7 +337,9 @@ static int decode_band_hdr(IVI4DecContext *ctx,
> > IVIBandDesc *band,
> >
> > band->is_empty = get_bits1(&ctx->gb);
> > if (!band->is_empty) {
> > - hdr_size = get_bits1(&ctx->gb) ? get_bits(&ctx->gb, 16) : 4;
> > + // skip header size
> > + if (get_bits1(&ctx->gb)) // if it is not given explicitly it's 4
> > + skip_bits(&ctx->gb, 16);
>
> The comment could be improved:
>
> /* skip header size
> * If header size is not given, header size is 4 bytes. */
>
committed with that comment change
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
