[MPlayer-dev-eng] autoexpand patch

Arpi arpi at mplayerhq.hu
Thu Nov 4 11:48:51 CET 2004


Hi,

> > It might seem like a hack, but how about just just using the "w" value 
> > as a decimal value?
> > For instance, -vf expand=1.33 would expand to a 4:3 ratio aspect...
> > 
> > How about that?
> 
> it's a pain. it could allow fraction (4/3, etc.) and that might be
> more useful.

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)

> another option is to make autoexpand a separate "filter", but using
> the same source file, just a different open() function and everything
> else the same. so vf_expand.c would export two filter definition
> structs almost identical...

it's ugly, and wont work in g2 or any future version supporting dynamic
plugin loading.


A'rpi / MPlayer, Astral & ESP-team

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




More information about the MPlayer-dev-eng mailing list