[MPlayer-dev-eng] autoexpand patch

[Dominik 'Rathann' Mierzejewski]

On Saturday, 13 November 2004 at 15:14, Romain Dolbeau wrote:
> Dominik 'Rathann' Mierzejewski wrote:
> 
> >It says: 720 576. Full mplayer -v log attached (I switched to fs and back
> >while playing).
> >
> >OTOH, I think the error may be in vo code, because of this line:
> >VO: [xv] 1024x767 => 1456x767 Planar YV12
> 
> That's normal behavior. the 720:576 DVD contains an aspect
> ratio (here 16/9, hence 1024:576) at which the DVD must be
> played.  Unfortunately, when you expand the image with
> -vf expand, the image is first expanded, and *then* the
> final display aspect ratio is computed.
> 
> So for a 720:576 image at 16:9, that's 1024:576 final, and the
> width will be expanded by (1024/720) ~= 1.42 You pad the
> image with black by asking for 1024 at 4:3, so the image is padded
> to 1024:767 (rounding error ?), and *then* the width is multiplied
> by ~1.42, and so display at 1456:767

In other words, it's not a bug, it's a feature? :(

> If you want a full-screen 4:3 picture with such DVD, you must
> expand to 720:768 (black bars top and bottom only), and let
> mplayer scale to 1024:768 during playing.

Hey, that's what we wanted to automatize, remember?

> If autoexpand could be fixed to properly handle the
> movie-with-embedded-aspect-ratio case, I'd be very ghappy :-)

Me too. ;)

R.

-- 
MPlayer RPMs maintainer: http://greysector.rangers.eu.org/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"