[MPlayer-dev-eng] [PATCH]Move calc_drwXY and fix indentation

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Fri Jan 9 11:00:56 CET 2009

On Fri, Jan 09, 2009 at 03:53:48AM +0100, Carl Eugen Hoyos wrote:
> Attached is a second try to move calc_drwXY from vo_xv (and vo_xvmc) into 
> x11_common.c, this time with hopefully correct indentation.

Ok for me. I tried to do this in a way that is more integrated and does
not need an extra function call, but I did not find a way yet and this
is much better than the current duplication.

> +        vo_dwidth = FFMIN(vo_dwidth, vo_screenwidth);
> +        vo_dheight = FFMIN(vo_dheight, vo_screenheight);
> +        *drwX = (vo_screenwidth - vo_dwidth) / 2;
> +        *drwY = (vo_screenheight - vo_dheight) / 2;

And yes, aligning these four lines while you're at it would be nice :-)

Reimar Döffinger

More information about the MPlayer-dev-eng mailing list