[MPlayer-dev-eng] [PATCH] Direct3D blackborders proper alignment

Diego Biurrun diego at biurrun.de
Mon Dec 8 12:08:25 CET 2008


On Mon, Dec 08, 2008 at 12:03:27PM +0200, Georgi Petrov wrote:
> --- libvo/vo_direct3d.c	(revision 28111)
> +++ libvo/vo_direct3d.c	(working copy)
> @@ -127,40 +127,65 @@
>      priv->fs_panscan_rect.bottom = priv->src_height;
> -    if (!vo_fs) return;
> +    if (!vo_fs && WinID < 0) return;

Please do not put multiple statements on one line.

> +    if (vo_fs)
> +    {
>  
> +        if (scaled_width > vo_dwidth) {
> +        if (scaled_height > vo_dheight) {
> +    if (!vo_fs && WinID >= 0)
> +    {
> +        if (w_asp > m_asp)
> +        {
> +        } else if (w_asp < m_asp)
> +        {

Please consistently keep braces on the same line as the if.

Diego



More information about the MPlayer-dev-eng mailing list