[FFmpeg-devel] [PATCH] Indeo 5 decoder
Sat Feb 6 10:07:10 CET 2010
On Thu, Feb 04, 2010 at 09:44:13AM +0200, Kostya wrote:
> On Wed, Feb 03, 2010 at 09:27:29PM +0100, Reimar D?ffinger wrote:
> > On Mon, Feb 01, 2010 at 07:49:55PM +0200, Kostya wrote:
> > > + uint32_t frame_num;
> > Only a 8-bit value is stored in that, also it is unused...
> still, it's in the bitstream. I'd like to keep it for debug purposes.
I wasn't objecting to reading it, I just don't like it much that
you're storing it in a context variable, that makes it rather hard
to find out it's not used.
Why do you think it is worse to do e.g.
> + ctx->frame_num = get_bits(&ctx->gb, 8);
skip_bits(&ctx->gb, 8); // frame_num
(or keep the get_bits, then it's even less change to re-add it).
More information about the ffmpeg-devel