[MPlayer-cvslog] r23154 - in trunk: DOCS/man/de/mplayer.1DOCS/man/en/mplayer.1 cfg-mplayer.h libvo/aspect.c

[Reimar Döffinger]

Hello,
On Thu, Oct 04, 2007 at 06:02:24AM +0300, Uoti Urpala wrote:
> On Wed, 2007-10-03 at 22:58 -0400, Rich Felker wrote:
> > > On Thu, 04 Oct 2007 02:57:43 +0200, Carl Eugen Hoyos scribed:
> > > >> Log:
> > > >> Make -monitorpixelaspect 0 the default.
> > > >
> > > >Unfortunately, this patch broke -vo aa. The picture is too narrow now.
> 
> > -monitorpixelaspect 0 is hardly even documented (what does "disables"
> > mean in the man page?!) and it's probably a bad default. Default
> > should be 1.
> 
> If you'd checked the actual commit you'd seen that the log message is
> wrong and it actually makes -monitorpixelaspect 1 the default. Obviously
> this is not a good default for aa, but I'm not sure whether we should
> care...

The main problem is that aalib has a bug that makes it break as soon as
we try to do aspect scaling. Unfortunately upstream seems to be quite
dead...
The right value for -monitorpixelaspect with aalib is of course
something like 8/15, depending on the font.

Greetings,
Reimar Döffinger