For the series,
Reviewed-by: Peter Harris <phar...@opentext.com>

One minor bikeshed-coloured nit:

On 2013-09-12 12:04, Adam Jackson wrote:
>              compRedirectWindow(serverClient, pWin, 
> CompositeRedirectAutomatic);
>              pWin->backStorage = (pointer) (intptr_t) 1;
> -        }
> -        else {
> +        } else {
>              compUnredirectWindow(serverClient, pWin,
>                                   CompositeRedirectAutomatic);
>              pWin->backStorage = NULL;

This hunk undoes the "-nce" flag passed to indent.
http://cgit.freedesktop.org/xorg/util/modular/tree/x-indent.sh#n12

I prefer -ce myself, so I'd rather see a patch to x-indent.sh than the
removal of this hunk, but I didn't speak up fast enough at the time.

Peter Harris
-- 
               Open Text Connectivity Solutions Group
Peter Harris                    http://connectivity.opentext.com/
Research and Development        Phone: +1 905 762 6001
phar...@opentext.com            Toll Free: 1 877 359 4866
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to