[Ffmpeg-devel] cropping/padding/rescaling patchset

[Luca Abeni]

Hi all,

this patchset rearranges the cropping, padding, and rescaling code in
ffmpeg to integrate swscale.

- The first patch is a modified version of Baptiste COUDURIER's padding
patch, that moves the padding code to img_pad() (in imgconvert.c) and
enables padding with pixel formats != yuv420p. I modified it to better
integrate it with my patches; I hope I did not introduce any bug. Credit
for the patch goes to Baptiste, bugs are mine.
(Baptiste, I hope you agree with my modifications... Let me know)

- The second patch modifies the cropping and padding code in ffmpeg.c
for making them independent from the rescaling code. After applying this
patch, imgresample.c only performs rescaling (the cropping and padding
code contained in it becomes superfluous).

- The third patch is purely cosmetic (the second patch introduced
indentation problems which are fixed by this patch)

- The fourth patch simplifies the code used for initializing video_crop,
video_pad, and video_resample

- The fifth patch provides support for external swscale

I did some quick tests, and everything seems to be ok. But there can be
bugs that I've not been able to trigger.

I hope the first 4 patches (or their cleaned up versions) can be
committed in CVS. The fifth patch is only for testing (swscale will be
ported to ffmpeg, so introducing support for it as an external library
does probably not make much sense).

I have a second version of this patchset, based on a different version
of the first patch (developed by me). I'll post it this evening or
tomorrow, for comparison.

			Thanks,
				Luca
-- 
_____________________________________________________________________________
Copy this in your signature, if you think it is important:
                               N O    W A R ! ! !
-- 
_____________________________________________________________________________
Copy this in your signature, if you think it is important:
                               N O    W A R ! ! !

 
 
 --
 Email.it, the professional e-mail, gratis per te: http://www.email.it/f
 
 Sponsor:
 DVD vergini: acquista online a prezzi vantaggiosi!
 Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=4589&d=20-3
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 01-padding-fix.diff
Type: text/x-patch
Size: 8744 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060320/f139653a/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 02-separation.diff
Type: text/x-patch
Size: 6020 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060320/f139653a/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 03-reindent.diff
Type: text/x-patch
Size: 2012 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060320/f139653a/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 04-simplify.diff
Type: text/x-patch
Size: 4630 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060320/f139653a/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 05-sws.diff
Type: text/x-patch
Size: 12831 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20060320/f139653a/attachment-0004.bin>