[FFmpeg-devel] [PATCH] Fix linesize computation for NV formats

Michael Niedermayer michaelni
Mon Nov 16 00:57:25 CET 2009

On Sun, Nov 15, 2009 at 11:23:27PM +0100, Stefano Sabatini wrote:
> On date Sunday 2009-11-15 17:13:22 +0100, Stefano Sabatini encoded:
> > Hi,
> > 
> > for NV12 and NV21 data[1] contains both the U and V components, so
> > its size needs to be multiplied by 2.
> > Alternatively we could have:
> > linesize[1] = width
> > 
> > which is equivalent.
> > 
> > I couldn't find any NV12 samples and libswscale cannot issue that in
> > output, so I didn't tested it.
> That change also requires a corresponding change in ff_fill_pointer(),
> so patch updated.
> I wonder if there is some reason if it has been done like that, it is
> done that way since the initial inclusion of NV formats support in
> r6115.

ill defer review of this until you claim that you confirmed a bug in
a test and that this patch fixes it.

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091116/e88c15ea/attachment.pgp>

More information about the ffmpeg-devel mailing list