wNETWMCheckInitialClientState is now void
authorRodolfo García Peñas (kix) <kix@kix.es>
Tue, 23 Oct 2012 21:59:47 +0000 (23 23:59 +0200)
committerCarlos R. Mafra <crmafra@gmail.com>
Sun, 28 Oct 2012 10:58:42 +0000 (28 10:58 +0000)
The function wNETWMCheckInitialClientState returns a Bool,
but this value is never used, so the function can be void.

kix@osaka:~/src/wmaker/wmaker-crm-tosnd/src$ grep wNETWMCheckInitialClientState *.[ch]
window.c:       wNETWMCheckInitialClientState(wwin);
wmspec.c:Bool wNETWMCheckInitialClientState(WWindow * wwin)
wmspec.c:       wmessage("wNETWMCheckInitialClientState");
wmspec.h:Bool wNETWMCheckInitialClientState(WWindow *wwin);
kix@osaka:~/src/wmaker/wmaker-crm-tosnd/src$

src/wmspec.c
src/wmspec.h

index f73f150..d2b335b 100644 (file)
@@ -1310,21 +1310,16 @@ static Bool updateNetIconInfo(WWindow * wwin)
        return hasState;
 }
 
-Bool wNETWMCheckInitialClientState(WWindow * wwin)
+void wNETWMCheckInitialClientState(WWindow *wwin)
 {
-       Bool hasState = False;
-
 #ifdef DEBUG_WMSPEC
        wmessage("wNETWMCheckInitialClientState");
 #endif
 
        wNETWMShowingDesktop(wwin->screen_ptr, False);
 
-       hasState |= updateNetIconInfo(wwin);
-
+       updateNetIconInfo(wwin);
        updateIconImage(wwin);
-
-       return hasState;
 }
 
 static void handleDesktopNames(XClientMessageEvent * event, WScreen * scr)
index e7438dc..fd70052 100644 (file)
@@ -32,7 +32,7 @@ void wNETWMInitStuff(WScreen *scr);
 void wNETWMCleanup(WScreen *scr);
 void wNETWMUpdateWorkarea(WScreen *scr, WArea usableArea);
 Bool wNETWMGetUsableArea(WScreen *scr, int head, WArea *area);
-Bool wNETWMCheckInitialClientState(WWindow *wwin);
+void wNETWMCheckInitialClientState(WWindow *wwin);
 Bool wNETWMProcessClientMessage(XClientMessageEvent *event);
 Bool wNETWMCheckClientHints(WWindow *wwin, int *layer, int *workspace);
 Bool wNETWMCheckClientHintChange(WWindow *wwin, XPropertyEvent *event);