Re: [hackers] [dmenu] [PATCH v2] Xinerama: correct variable declarations in preprocessor conditional

From: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
Date: Fri, 25 Nov 2016 14:24:41 +0100

On Fri, Nov 25, 2016 at 10:38:09PM +1000, Thomas Gardner wrote:
> ---
> Sorry Quentin, I know now that this is where to include remarks.
>
> dmenu.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/dmenu.c b/dmenu.c
> index 9278e91..1cfefcd 100644
> --- a/dmenu.c
> +++ b/dmenu.c
> _AT_@ -539,15 +539,16 @@ run(void)
> static void
> setup(void)
> {
> - int x, y;
> + int x, y, i = 0;
> + unsigned int du;
> XSetWindowAttributes swa;
> XIM xim;
> + Window w, dw, *dws;
> + XWindowAttributes wa;
> #ifdef XINERAMA
> XineramaScreenInfo *info;
> - Window w, pw, dw, *dws;
> - XWindowAttributes wa;
> - int a, j, di, n, i = 0, area = 0;
> - unsigned int du;
> + Window pw;
> + int a, j, di, n, area = 0;
> #endif
>
> /* init appearance */
> --
> 2.10.2
>

Hey Thomas,

Thanks for the patch. I have pushed it to master.

-- 
Kind regards,
Hiltjo
Received on Fri Nov 25 2016 - 14:24:41 CET

This archive was generated by hypermail 2.3.0 : Fri Nov 25 2016 - 14:36:18 CET