config_path cleanup

sway/config.c

@@ -13,20 +13,23 @@
 
 struct sway_config *config;
 
-static bool exists(const char *path) {
+static bool file_exists(const char *path) {
 	return access(path, R_OK) != -1;
 }
 
-void config_defaults(struct sway_config *config) {
+static void config_defaults(struct sway_config *config) {
 	config->symbols = create_list();
 	config->modes = create_list();
-	config->cmd_queue = create_list();
 	config->workspace_outputs = create_list();
 	config->output_configs = create_list();
+
+	config->cmd_queue = create_list();
+
 	config->current_mode = malloc(sizeof(struct sway_mode));
 	config->current_mode->name = NULL;
 	config->current_mode->bindings = create_list();
 	list_add(config->modes, config->current_mode);
+
 	config->floating_mod = 0;
 	config->default_layout = L_NONE;
 	config->default_orientation = L_NONE;
@@ -44,7 +47,14 @@ void config_defaults(struct sway_config *config) {
 
 void free_mode(struct sway_mode *mode) {
 	free(mode->name);
-	free_flat_list(mode->bindings);
+	int i;
+	for (i = 0; i < mode->bindings->length; ++i) {
+		struct sway_binding *bind = mode->bindings->items[i];
+		list_free(bind->keys);
+		free(bind->command);
+		free(bind);
+	}
+	list_free(mode->bindings);
 }
 
 void free_config(struct sway_config *config) {
@@ -69,88 +79,74 @@ void free_config(struct sway_config *config) {
 	free_flat_list(config->output_configs);
 }
 
-static const char *search_paths[] = {
-	"$home/.sway/config",
-	"$config/sway/config",
-	"/etc/sway/config",
-	"$home/.i3/config",
-	"$config/.i3/config",
-	"/etc/i3/config"
-};
 
-static char *get_config_path() {
-	char *home = getenv("HOME");
-	if (home) {
-		home = strdup(getenv("HOME"));
-	}
-	char *config = getenv("XDG_CONFIG_HOME");
-	if (config) {
-		config = strdup(getenv("XDG_CONFIG_HOME"));
+static char *get_config_path(void) {
+	char *config_path = NULL;
+	char *paths[3] = {getenv("HOME"), getenv("XDG_CONFIG_HOME"), ""};
+	int pathlen[3] = {0, 0, 0};
+	int i;
+#define home paths[0]
+#define conf paths[1]
+	// Get home and config directories
+	home = home ? strdup(home) : NULL;
+	if (conf) {
+		conf = strdup(conf);
 	} else if (home) {
 		const char *def = "/.config";
-		config = malloc(strlen(home) + strlen(def) + 1);
-		strcpy(config, home);
-		strcat(config, def);
+		conf = malloc(strlen(home) + strlen(def) + 1);
+		strcpy(conf, home);
+		strcat(conf, def);
 	} else {
 		home = strdup("");
-		config = strdup("");
+		conf = strdup("");
 	}
-
-	// Set up a temporary config for holding set variables
-	struct sway_config *temp_config = malloc(sizeof(struct sway_config));
-	config_defaults(temp_config);
-	const char *set_home = "set $home ";
-	char *_home = malloc(strlen(home) + strlen(set_home) + 1);
-	strcpy(_home, set_home);
-	strcat(_home, home);
-	handle_command(temp_config, _home);
-	free(_home);
-	const char *set_config = "set $config ";
-	char *_config = malloc(strlen(config) + strlen(set_config) + 1);
-	strcpy(_config, set_config);
-	strcat(_config, config);
-	handle_command(temp_config, _config);
-	free(_config);
-
-	char *test = NULL;
-	int i;
+	pathlen[0] = strlen(home);
+	pathlen[1] = strlen(conf);
+#undef home
+#undef conf
+	// Search for config file from search paths
+	static const char *search_paths[] = {
+		"/.sway/config", // Prepend with $home
+		"/sway/config", // Prepend with $config
+		"/etc/sway/config",
+		"/.i3/config", // $home
+		"/.i3/config", // $config
+		"/etc/i3/config"
+	};
 	for (i = 0; i < (int)(sizeof(search_paths) / sizeof(char *)); ++i) {
-		test = strdup(search_paths[i]);
-		test = do_var_replacement(temp_config, test);
+		char *test = malloc(pathlen[i%3] + strlen(search_paths[i]) + 1);
+		strcpy(test, paths[i%3]);
+		strcat(test, search_paths[i]);
 		sway_log(L_DEBUG, "Checking for config at %s", test);
-		if (exists(test)) {
-			goto _continue;
+		if (file_exists(test)) {
+			config_path = test;
+			goto cleanup;
 		}
 		free(test);
-		test = NULL;
 	}
 
 	sway_log(L_DEBUG, "Trying to find config in XDG_CONFIG_DIRS");
 	char *xdg_config_dirs = getenv("XDG_CONFIG_DIRS");
-	if (xdg_config_dirs != NULL) {
+	if (xdg_config_dirs) {
 		list_t *paths = split_string(xdg_config_dirs, ":");
-		char *name = "/sway/config";
-		int i;
+		const char *name = "/sway/config";
 		for (i = 0; i < paths->length; i++ ) {
-			test = malloc(strlen(paths->items[i]) + strlen(name) + 1);
+			char *test = malloc(strlen(paths->items[i]) + strlen(name) + 1);
 			strcpy(test, paths->items[i]);
 			strcat(test, name);
-			if (exists(test)) {
-				free_config(temp_config);
-				free_flat_list(paths);
-				return test;
+			if (file_exists(test)) {
+				config_path = test;
+				break;
 			}
 			free(test);
-			test = NULL;
 		}
 		free_flat_list(paths);
 	}
 
-_continue:
-	free_config(temp_config);
-	free(home);
-	free(config);
-	return test;
+cleanup:
+	free(paths[0]);
+	free(paths[1]);
+	return config_path;
 }
 
 bool load_config(const char *file) {

sway/handlers.c

@@ -334,7 +334,6 @@ static bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifier
 		release_key(sym, key);
 	}
 
-	// TODO: reminder to check conflicts with mod+q+a versus mod+q
 	for (i = 0; i < mode->bindings->length; ++i) {
 		struct sway_binding *binding = mode->bindings->items[i];