[PATCH 17/37] misc: Fold together some redundant conditionals

Alan Coopersmith alan.coopersmith at oracle.com
Thu Oct 9 05:47:45 PDT 2014


On 10/ 8/14 05:04 PM, Adam Jackson wrote:
> @@ -2576,10 +2576,9 @@ MapWindow(WindowPtr pWin, ClientPtr client)
>               if (anyMarked) {
>                   (*pScreen->ValidateTree) (pLayerWin->parent, pLayerWin, VTMap);
>                   (*pScreen->HandleExposures) (pLayerWin->parent);
> -            }
> -            if (anyMarked && pScreen->PostValidateTree)
> -                (*pScreen->PostValidateTree) (pLayerWin->parent, pLayerWin,
> -                                              VTMap);
> +                if (pScreen->PostValidateTree)
> +                    (*pScreen->PostValidateTree) (pLayerWin->parent, pLayerWin,
> +                                                  VTMap);
>           }
>           WindowsRestructured();
>       }

Am I missing something or did you lose the closing } for the if (anyMarked) body
in this hunk?

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list