[MPlayer-dev-eng] autoexpand patch

Arpi arpi at mplayerhq.hu
Fri Nov 5 16:20:44 CET 2004


Hi,

> > why not add a new (6th) option to expand, so someone could do things
> > like:
> > -vf expand=:600:::4/3  to get the video expanded to 600 height, and
> > the width calculated from the given 4:3 aspect ratio.
> > also if someone only wants aspect set, do -vf expand=aspect=4/3
> > (it's handled by subconfig parser, equal to expand=:::::4/3)
> 
> Hello.
> 
> I have inserted aspect option to vf_expand.c as suggested here, with one
> exception: aspect is not defined like x/y, but there are 2 separate
> parameters named ratio_x and ratio_y. Functionality is the same, only
> "/" did problems with buitin checks on parameters, because it expected
> it to be one integer value. Maybe there was some workaround (type of
> parameter other than integer), but I am not too familiar with mplayer
> source code, so I simply split it.

as 4/3 == 1.333, it's not an integer of course. set type to float, and
it will work.


A'rpi / MPlayer, Astral & ESP-team

--
MPlayer's new image: happiness & peace & cosmetics & vmiklos




More information about the MPlayer-dev-eng mailing list