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

Stefano Sabatini stefano.sabatini-lala
Fri Mar 5 01:53:31 CET 2010


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...

Regards.
-- 
FFmpeg = Furious Fundamentalist Mystic Powerful Entertaining Ghost



More information about the ffmpeg-devel mailing list