Make swaybar run without status_command

swaybar/main.c

@@ -189,7 +189,9 @@ void bar_ipc_init(int outputi, const char *bar_id) {
 
 	// TODO: More of these options
 	// TODO: Refactor swaybar into several files, create a bar config struct (shared with compositor?)
-	if (_status_command) status_command = strdup(json_object_get_string(_status_command));
+	if (_status_command) {
+		status_command = strdup(json_object_get_string(_status_command));
+	}
 
 	if (_colors) {
 		json_object *background, *statusline, *separator;
@@ -219,31 +221,43 @@ void bar_ipc_init(int outputi, const char *bar_id) {
 		if (background) colors.background = parse_color(json_object_get_string(background));
 		if (statusline) colors.statusline = parse_color(json_object_get_string(statusline));
 		if (separator) colors.seperator = parse_color(json_object_get_string(separator));
-		if (focused_workspace_border)
+		if (focused_workspace_border) {
 			colors.focused_workspace.border = parse_color(json_object_get_string(focused_workspace_border));
-		if (focused_workspace_bg)
+		}
+		if (focused_workspace_bg) {
 			colors.focused_workspace.background = parse_color(json_object_get_string(focused_workspace_bg));
-		if (focused_workspace_text)
+		}
+		if (focused_workspace_text) {
 			colors.focused_workspace.text = parse_color(json_object_get_string(focused_workspace_text));
-		if (active_workspace_border)
+		}
+		if (active_workspace_border) {
 			colors.active_workspace.border = parse_color(json_object_get_string(active_workspace_border));
-		if (active_workspace_bg)
+		}
+		if (active_workspace_bg) {
 			colors.active_workspace.background = parse_color(json_object_get_string(active_workspace_bg));
-		if (active_workspace_text)
+		}
+		if (active_workspace_text) {
 			colors.active_workspace.text = parse_color(json_object_get_string(active_workspace_text));
-		if (inactive_workspace_border)
+		}
+		if (inactive_workspace_border) {
 			colors.inactive_workspace.border = parse_color(json_object_get_string(inactive_workspace_border));
-		if (inactive_workspace_bg)
+		}
+		if (inactive_workspace_bg) {
 			colors.inactive_workspace.background = parse_color(json_object_get_string(inactive_workspace_bg));
-		if (inactive_workspace_text)
+		}
+		if (inactive_workspace_text) {
 			colors.inactive_workspace.text = parse_color(json_object_get_string(inactive_workspace_text));
-		if (binding_mode_border)
+		}
+		if (binding_mode_border) {
 			colors.binding_mode.border = parse_color(json_object_get_string(binding_mode_border));
-		if (binding_mode_bg)
+		}
+		if (binding_mode_bg) {
 			colors.binding_mode.background = parse_color(json_object_get_string(binding_mode_bg));
-		if (binding_mode_text)
+		}
+		if (binding_mode_text) {
 			colors.binding_mode.text = parse_color(json_object_get_string(binding_mode_text));
 		}
+	}
 
 	json_object_put(bar_config);
 	free(res);
@@ -258,6 +272,8 @@ void bar_ipc_init(int outputi, const char *bar_id) {
 
 void update() {
 	int pending;
+	// If no command is set, we don't have to update anything
+	if (status_command) {
 		if (ioctl(fileno(command), FIONREAD, &pending) != -1 && pending > 0) {
 			free(line);
 			line = read_line(command);
@@ -266,6 +282,7 @@ void update() {
 				line[l] = '\0';
 			}
 		}
+	}
 	if (ioctl(socketfd, FIONREAD, &pending) != -1 && pending > 0) {
 		sway_log(L_INFO, "data available");
 		uint32_t len;
@@ -396,9 +413,11 @@ int main(int argc, char **argv) {
 
 	bar_ipc_init(desired_output, bar_id);
 
+	if (status_command) {
 		command = popen(status_command, "r");
 		line = malloc(1024);
 		line[0] = '\0';
+	}
 
 	window = window_setup(registry, output->width, 30, false);
 	if (!window) {