[FFmpeg-devel] MXF D10 regression tests

[Reimar Döffinger]

On Thu, Mar 19, 2009 at 12:48:49AM +0100, Michael Niedermayer wrote:
> On Thu, Mar 19, 2009 at 12:20:51AM +0100, Reimar D?ffinger wrote:
> > On Wed, Mar 18, 2009 at 11:35:26PM +0100, Michael Niedermayer wrote:
> > > > +        ff_fill_linesize(&picture, s->pix_fmt, w, stride_align);
> > > > +
> > > 
> > > does something like:
> > > retry:
> > >     ff_fill_linesize(&picture, s->pix_fmt, w);
> > >     if(any linesize % stride_align){
> > >         w+= w&~(w-1);
> > >         goto retry;
> > >     }
> > > 
> > > work?
> > 
> > I can say for sure it would be a bit more ugly, like this it would be an
> > endless loop for the paletted etc. formats (though if it makes sense too
> > set a linesize for the palette is a different question).
> 
> ?
> pal with linesize=0 would have if(0%16) and that should not loop endlessly
> am i missing something?

Your missing that PAL8 currently has linesize[1] = 4