[FFmpeg-devel] [PATCH 1/2] vf_scale: add force_original_aspect_ratio

[Michael Niedermayer]

On Sat, Aug 03, 2013 at 05:15:35PM +0300, Andrey Utkin wrote:
> 2013/8/3 Stefano Sabatini <stefasab at gmail.com>:
> >> +
> >> +One useful instance of this option is that when you know a specific device's
> >> +maximum allowed resolution, you can use this to limit the output video to
> >> +that, while retaining the aspect ratio. For example, device A allows
> >> +1280x720 playback, and your video is 1920x800. Using this option (set it to
> >> +1) and specifying 1280x720 to the command line makes the output 1280x533.
> >
> > Not sure this is a good idea, since this tricks the output size in
> > possibly unexpected ways. Why can't you use an expression to set that
> > condition, so that outw/h depends on the original input aspect ratio?
> 
> I am also unsure on usefulness of this feature. To add to the above,
> on such resizes user is possible to want dimensions to be divisable by
> 2, 4 or 8...

a align=X option could be added that aligns the dimensions to the
closest multiple of X, this could be usefull independant of the cases
discussed in this thread


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130804/8165723c/attachment.asc>