Remove the phase2 variable

"Rodolfo García Peñas (kix)" <[email protected]>
Newsgroups gmane.compw.window-managers.windowmaker.devel
Message-ID <[email protected]>
Hi,


The variable phase2 can be removed. Is only used in one place. This  
patch is for awmaker, but probably it could be adapted to wmaker.

Cheers,
kix
Rodolfo García Peñas (kix)
http://www.kix.es/
0001-Removed-w_global.startup.phase2-variable.patch (text/x-diff, 2.1 KB)
From 658e5cd52c791b1e0e56c2f20c3d13f745737fcb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rodolfo=20Garc=C3=ADa=20Pe=C3=B1as=20=28kix=29?=
 <[email protected]>
Date: Mon, 20 Jul 2015 00:50:45 +0200
Subject: [PATCH] Removed w_global.startup.phase2 variable

The variable w_global.startup.phase2 can be removed because is used only
in the function wWorkspaceForceChange, so we can remove the call and the
if about this variable.
---
 src/WindowMaker.h | 1 -
 src/startup.c     | 5 -----
 src/workspace.c   | 2 +-
 3 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/WindowMaker.h b/src/WindowMaker.h
index 1f466a6..a72124a 100644
--- a/src/WindowMaker.h
+++ b/src/WindowMaker.h
@@ -502,7 +502,6 @@ extern struct wmaker_global_variables {
 	/* Window Maker startup phases */
 	struct {
 		unsigned int phase1:1;
-		unsigned int phase2:1;
 	} startup;
 
 	/* locale to use. NULL==POSIX or C */
diff --git a/src/startup.c b/src/startup.c
index decea51..23f4cef 100644
--- a/src/startup.c
+++ b/src/startup.c
@@ -695,8 +695,6 @@ void StartUp(Bool defaultScreenOnly)
 
 		manageAllWindows(w_global.vscreens[j], wPreferences.flags.restarting == 2);
 
-		w_global.startup.phase2 = 1;
-
 		while (XPending(dpy)) {
 			XEvent ev;
 			WMNextEvent(dpy, &ev);
@@ -704,12 +702,9 @@ void StartUp(Bool defaultScreenOnly)
 		}
 
 		vscr->workspace.last_used = 0;
-		wWorkspaceForceChange(vscr, 0);
 		if (!wPreferences.flags.noclip)
 			wDockShowIcons(vscr->workspace.array[vscr->workspace.current]->clip);
 
-		w_global.startup.phase2 = 0;
-
 		/* restore saved menus */
 		wMenuRestoreState(w_global.vscreens[j]);
 
diff --git a/src/workspace.c b/src/workspace.c
index f4ff4a0..596ba8f 100644
--- a/src/workspace.c
+++ b/src/workspace.c
@@ -540,7 +540,7 @@ static void showWorkspaceName(virtual_screen *vscr, int workspace)
 
 void wWorkspaceChange(virtual_screen *vscr, int workspace)
 {
-	if (w_global.startup.phase1 || w_global.startup.phase2 || vscr->screen_ptr->flags.ignore_focus_events)
+	if (w_global.startup.phase1 || vscr->screen_ptr->flags.ignore_focus_events)
 		return;
 
 	if (workspace != vscr->workspace.current)
-- 
2.1.4
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.