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