From 646e3ce242a00687877259d3b2d7866e0f42f2fb Mon Sep 17 00:00:00 2001
From: Ulrich Sibiller <uli42@gmx.de>
Date: Thu, 3 Oct 2019 23:39:01 +0200
Subject: Screen.c: cleanup code a bit

---
 nx-X11/programs/Xserver/hw/nxagent/Screen.c | 30 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 16 deletions(-)

(limited to 'nx-X11/programs/Xserver')

diff --git a/nx-X11/programs/Xserver/hw/nxagent/Screen.c b/nx-X11/programs/Xserver/hw/nxagent/Screen.c
index ae64c0a18..2b1146496 100644
--- a/nx-X11/programs/Xserver/hw/nxagent/Screen.c
+++ b/nx-X11/programs/Xserver/hw/nxagent/Screen.c
@@ -418,27 +418,23 @@ FIXME: We'll check for ReparentNotify and LeaveNotify events after
 
 Window nxagentCreateIconWindow(void)
 {
-  XSetWindowAttributes attributes;
-  unsigned long valuemask;
-  XSizeHints* sizeHints;
-  XWMHints* wmHints;
-  Window w;
-  Mask mask;
-
   /*
    * Create icon window.
    */
 
-  attributes.override_redirect = False;
-  attributes.colormap = DefaultColormap(nxagentDisplay, DefaultScreen(nxagentDisplay));
-  attributes.background_pixmap = nxagentScreenSaverPixmap;
-  valuemask = CWOverrideRedirect | CWBackPixmap | CWColormap;
+  XSetWindowAttributes attributes = {
+    .override_redirect = False,
+    .colormap = DefaultColormap(nxagentDisplay, DefaultScreen(nxagentDisplay)),
+    .background_pixmap = nxagentScreenSaverPixmap,
+  };
+
+  unsigned long valuemask = CWOverrideRedirect | CWBackPixmap | CWColormap;
 
   #ifdef TEST
   fprintf(stderr, "nxagentCreateIconWindow: Going to create new icon window.\n");
   #endif
 
-  w = XCreateWindow(nxagentDisplay, DefaultRootWindow(nxagentDisplay),
+  Window w = XCreateWindow(nxagentDisplay, DefaultRootWindow(nxagentDisplay),
                         0, 0, 1, 1, 0,
                             DefaultDepth(nxagentDisplay, DefaultScreen(nxagentDisplay)),
                                 InputOutput,
@@ -458,14 +454,17 @@ Window nxagentCreateIconWindow(void)
    *  Set hints to the window manager for the icon window.
    */
 
-  if ((sizeHints = XAllocSizeHints()))
+  XSizeHints* sizeHints = XAllocSizeHints();
+  XWMHints* wmHints = XAllocWMHints();;
+
+  if (sizeHints)
   {
     sizeHints->flags = PMinSize | PMaxSize;
     sizeHints->min_width = sizeHints->max_width = 1;
     sizeHints->min_height = sizeHints->max_height = 1;
   }
 
-  if ((wmHints = XAllocWMHints()))
+  if (wmHints)
   {
     wmHints->flags = IconPixmapHint | IconMaskHint;
     wmHints->initial_state = IconicState;
@@ -483,7 +482,6 @@ Window nxagentCreateIconWindow(void)
   }
 
   char *window_name = nxagentWindowName;
-
   Xutf8SetWMProperties(nxagentDisplay, w,
                       window_name, window_name,
                           NULL , 0 , sizeHints, wmHints, NULL);
@@ -495,7 +493,7 @@ Window nxagentCreateIconWindow(void)
    * Enable events from the icon window.
    */
 
-  mask = nxagentGetDefaultEventMask();
+  Mask mask = nxagentGetDefaultEventMask();
 
   XSelectInput(nxagentDisplay, w, (mask & ~(KeyPressMask |
                    KeyReleaseMask)) | StructureNotifyMask);
-- 
cgit v1.2.3