[FFmpeg-devel] [PATCH] Scale width/height keeping aspect when given just one dimension

[Daniel G. Taylor]

On Sun, 2009-09-27 at 22:14 +0200, Michael Niedermayer wrote:
> -s will be replaced by libavfilters scale filter.
> i will not review this and am mildly against it being applied because
> every improvment done to the current stuff means there will be someone
> who will cry foul if its not also in libavfilter when we switch ->
> which will delay switching or make it a bigger mess than needed ...

Understandable, however fwiw this patch is basically a port _from_
libavfilter/vf_scale.c (see lines 111, 112 in soc repo) which already
supports the feature. The syntax and behavior is identical. This just
adds the feature to FFmpeg while we wait for libavfilter to get into
trunk properly.

If you still don't want this patch in trunk that's fine, just throwing
it out there as others may find it useful and libavfilter still seems to
have some issues. I've seen quite a few requests for it in #ffmpeg and
always wind up telling users to scan the output of ffmpeg -i myfile.mp4,
scale the values manually, then invoke the actual transcoding command.
Now at least I can point to this patch.

Take care,
-- 
Daniel G. Taylor
http://programmer-art.org/