[Ffmpeg-devel] Re: Cropping / Padding patches

Michael Niedermayer michaelni
Tue Mar 14 22:45:07 CET 2006


On Tue, Mar 14, 2006 at 09:05:16AM +0100, Luca Abeni wrote:
> > I suggest moving fill_pad_region to img_pad in imgconvert.c
> This looks reasonable. I also propose to change the function prototype
> so that it takes an additional AVPicture* (or AVFrame*, I do not know)
> parameter, like the picture_pad_temp used in my patch.
> In this way, you can pad by
> 1) calling img_pad(final_picture, final_height, final_width, padtop, padbottom, padleft, padright, padcolor, &picture_pad_temp)
> 2) copying (or rescaling) the input picture into picture_pad_temp
> 3) the padded picture is in final_picture
> Michael, what do you think? Is this reasonable?

my interrest in the scale/crop/pad code in and related to ffmoeg.c is zero
in its current form, so do whatever you think is best ... when we have a 
clean video filter layer then ill be more interrested for sure ...


More information about the ffmpeg-devel mailing list