[FFmpeg-devel] [PATCH] MPEG sequeunce display extension parsing off by 2 error
Tue Jun 29 04:29:48 CEST 2010
On Tue, 2010-06-29 at 01:28 +0200, Michael Niedermayer wrote:
> On Mon, Jun 28, 2010 at 01:09:02PM -0400, Daniel Kristjansson wrote:
> > This isn't a very important patch since these are trailing bits,
> > but if we're going to have a skip_bits call it should be skipping
> > the correct number of bits.
> > - skip_bits(&s->gb, 1); //marker
> > + skip_bits(&s->gb, 3); //marker
> my copy of the spec doesnt contain 3 marker bits
> could you quote the relevant part of the spec please (including
> which exact version that is)?
13818-2: 1995(E), p33. They are not actually marker bits, they are
undefined bits, so they do not have to be set like the bit between
the horizontal and vertical size.
More information about the ffmpeg-devel