[MPlayer-dev-eng] [PATCH] use standard aspect functions for vo_quartz

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Aug 28 10:34:56 CEST 2009


On Fri, Aug 28, 2009 at 10:15:41AM +0200, Guillaume POIRIER wrote:
> 2009/8/28 Reimar Döffinger <Reimar.Doeffinger at gmx.de>:
> > vo_quartz currently reimplements the whole aspect handling (and probably
> > badly, too), attached patch changes it to use the standard functions
> > where obvious.
> > I realized there are some other things missing like -geometry support
> > that at least after this change should work for specifying size, but
> > position is ignored.
> > Panscan is also still handled in an ugly way that can only work with
> > fullscreen windows and even there leaves ugly artefacts on the
> > neighbouring displays if there are any I think.
> 
> Tested and works as expected on my machine.
> Feel free to do whatever you want with vo_quartz: this VO won't be
> developed further anyway since it uses Carbon, which isn't 64-bits
> compatible. That means that it won't build by default on Snow Leopard.

I'd like to point out that vo_corevideo uses exactly the same code, so
most of this should be done there (and some extracted into a shared
file)...



More information about the MPlayer-dev-eng mailing list