[FFmpeg-devel] [PATCH] MPEG sequeunce display extension parsing off by 2 error

[Daniel Kristjansson]

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.

-- Daniel