[FFmpeg-devel] half vertical chroma resolution from JPEGs..

Michael Niedermayer michaelni
Tue Jul 17 23:52:15 CEST 2007


On Sat, Jul 14, 2007 at 11:36:38AM +0200, Andreas ?man wrote:
> Hi
> Michael Niedermayer wrote:
> >Hi
> >
> >see the thread about " [PATCH] MJPEG wrongly displayed with vertical 
> >YUV422"
> Ah, thanks
> >also the patch is ok only if it works with swscaler too
> >
> It works, but when dealing with large images, swscale seem
> to crash (i doubt this is related to yuv440 images only)
> If I compile without libswscale, it works fine.
> After some valgrinding (see below) i suspected the
> c->lumPixBuf[i]= c->lumPixBuf[i+c->vLumBufSize]= av_mallocz(4000);
> -line. Just setting it to 8000 makes it work all fine, but i
> believe the real solution is to compute this value, but how?
> I dont know the swscale internals enough to figure that out.

it should be (dstW+1)*sizeof(int16_t) _IIRC_ (totally untested!)

and patch looks ok

also maybe you want to add the new format to tests/regression.sh ...


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070717/5e721a26/attachment.pgp>

More information about the ffmpeg-devel mailing list