[FFmpeg-devel] [PATCH] Re-implement ff_fill_linesize() using pixdescs info

[Stefano Sabatini]

On date Friday 2010-03-05 01:53:31 +0100, Stefano Sabatini encoded:
> On date Friday 2010-03-05 01:42:01 +0100, Vitor Sessak encoded:
> > Stefano Sabatini wrote:
> > >Hi, as discussed here:
> > >http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/99224/focus=99279
> > >
> > >The idea is to make simpler/more general the code of ff_fill_linesize(),
> > >I have a similar patch for ff_fill_pointer().
> > >
> > >The test shows some difference between the old and the new
> > >implementation for odd sizes, the new implementation looks more
> > >correct in all cases.
> > >
> > >In attachment the test program and the patch meant for review.
> > 
> > [...]
> > 
> > >
> > >Mismatch for pixel format 'yuv422p16le' with size 1
> > >ff_fill_linesize : 2 2 2 0
> > >ff_fill_linesize2: 2 1 1 0
> > 
> > And I missing something or linesize for all 16-bit formats should be
> > always even?
> 
> Yes so the code needs to be readjusted...

Updated.

The differences with the old version looks always correct now, code is
about 1/4 of the original one (slightly slower), and it is general
enough so new format additions should not require changes to it.

Patch 0001 implementing a test program attached only for reference.

Regards.
-- 
FFmpeg = Fierce and Free Moronic Picky Ecletic Glue