[MPlayer-dev-eng] Re: [PATCH] Pivoting

[Rich Felker]

On Wed, Aug 24, 2005 at 04:11:56PM +0200, Pawe? Sakowski wrote:
> OK, another approach:
> 
> - mplayer-pivot.patch: to detect the rotation, I use libXrandr directly.
> This introduces a library dependency, but (as per the library design)
> does not require support in the X server. Again, I don't know how RandR
> interacts with Xinerama, so I'm not changing the behavior of the latter.
> 
> - mplayer-aspect.patch: monitor_aspect is taken from (in the order of
> precedence):
> 1. explicit specification in config or command-line
> 2. physical width/height as reported by vo (currently only implemented
> for X11-based vo, using DisplayWidthMM)
> 3. square pixel assumption
> 
> Display{Width,Height}MM seem always to return sane values. If there's no
> authoritative source of the display's size, they simply calculate it
> from the pixel count and some pressumed dpi.

Can we include rounding to make sure the aspect is sane? I.e. if it's
within some epsilon of the ratio of pixels, replace it by the ratio of
pixels?

Rich