[MPlayer-dev-eng] [PATCH] -aspect-tolerance option to avoid tiny prescaling

Kurt Garloff kurt at garloff.de
Mon Mar 31 10:59:47 CEST 2008


Attila,

On Fri, Mar 28, 2008 at 02:08:38PM +0100, Attila Kinali wrote:
> On Fri, 28 Mar 2008 13:51:07 +0100
> Kurt Garloff <kurt at garloff.de> wrote:
> 
> > In general I think we should avoid scaling if the difference in
> > aspect ratio is tiny.
> > 
> > Attached patch does this for AVI files.
> 
> This code belongs into libvo/aspect.c
> 
> You should also make the limit configurable by adding a command
> line option. It should be also possible to switch this "protection"
> off (in some cases, the resulting black bars are more annoying).

Thanks for the good feedback!

I could not find a clean way to add the logic to libvo/aspect.c, as
the prescaling is configured before that code gets called.

A new patch attached, putting the logic into libmpcodecs/vd.c, making
it adjustable (-aspect-tolerance FLOAT) and documenting it.

Best,
-- 
Kurt Garloff                   <kurt at garloff.de>             [Koeln, DE]
Physics:Plasma modeling <garloff at plasimo.phys.tue.nl> [TU Eindhoven, NL]
Linux: VP PM OPS               <garloff at suse.de>            [Novell Inc]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-aspect-tolerance.diff
Type: text/x-patch
Size: 3355 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080331/12fba45d/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 194 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080331/12fba45d/attachment.pgp>


More information about the MPlayer-dev-eng mailing list