Fixes to workspace generation

2 files changed, 14 insertions, 11 deletions

diff --git a/sway/main.c b/sway/main.c
index 7477b08c..b48d4b19 100644
--- a/sway/main.c
+++ b/sway/main.c
@@ -18,6 +18,9 @@ int main(int argc, char **argv) {
 	/* Signal handling */
 	signal(SIGCHLD, sigchld_handle);
 
+	if (!load_config()) {
+		sway_abort("Unable to load config");
+	}
 
 	setenv("WLC_DIM", "0", 0);
 	if (!wlc_init(&interface, argc, argv)) {
@@ -25,10 +28,6 @@ int main(int argc, char **argv) {
 	}
 	setenv("DISPLAY", ":1", 1);
 
-	if (!load_config()) {
-		sway_abort("Unable to load config");
-	}
-
 	wlc_run();
 	return 0;
 }
diff --git a/sway/workspace.c b/sway/workspace.c
index 96604678..9bc6838c 100644
--- a/sway/workspace.c
+++ b/sway/workspace.c
@@ -6,22 +6,27 @@
 #include "list.h"
 #include "log.h"
 #include "container.h"
+#include "config.h"
+#include "stringop.h"
 
 swayc_t *active_workspace = NULL;
 
-int ws_num = 1;
-
 char *workspace_next_name(void) {
+	sway_log(L_DEBUG, "Workspace: Generating new name");
 	int i;
+    int l = 1;
     // Scan all workspace bindings to find the next available workspace name,
     // if none are found/available then default to a number
 	struct sway_mode *mode = config->current_mode;
 
 	for (i = 0; i < mode->bindings->length; ++i) {
-		const char* command = *binding = mode->bindings->items[i]->command;
+		struct sway_binding *binding = mode->bindings->items[i];
+		const char* command = binding->command;
 	    list_t *args = split_string(command, " ");
+	    sway_log(L_DEBUG, "Workspace: Checking name '%s'", command);
 
-        if (strcmp("workspace", args->items[0]) == 0 && args->length > 2) {
+        if (strcmp("workspace", args->items[0]) == 0 && args->length > 1) {
+	        sway_log(L_DEBUG, "Got valid workspace command for target: '%s'", args->items[1]);
             const char* target = args->items[1];
 
             while (*target == ' ' || *target == '\t')
@@ -39,11 +44,10 @@ char *workspace_next_name(void) {
                 continue;
            
             //Make sure that the workspace doesn't already exist 
-            if (workspace_find_by_name(args->items[2]))
+            if (workspace_find_by_name(args->items[1]))
                continue; 
 
-            return args->items[2];        
-        }
+            return args->items[1];        }
     }
     // As a fall back, get the current number of active workspaces
     // and return that + 1 for the next workspace's name