aboutsummaryrefslogtreecommitdiff
path: root/swaybar
diff options
context:
space:
mode:
Diffstat (limited to 'swaybar')
-rw-r--r--swaybar/bar.c112
-rw-r--r--swaybar/i3bar.c17
-rw-r--r--swaybar/render.c5
3 files changed, 88 insertions, 46 deletions
diff --git a/swaybar/bar.c b/swaybar/bar.c
index 3ae730f7..69069f40 100644
--- a/swaybar/bar.c
+++ b/swaybar/bar.c
@@ -48,8 +48,13 @@ static void swaybar_output_free(struct swaybar_output *output) {
return;
}
wlr_log(WLR_DEBUG, "Removing output %s", output->name);
- zwlr_layer_surface_v1_destroy(output->layer_surface);
- wl_surface_destroy(output->surface);
+ if (output->layer_surface != NULL) {
+ zwlr_layer_surface_v1_destroy(output->layer_surface);
+ }
+ if (output->surface != NULL) {
+ wl_surface_destroy(output->surface);
+ }
+ zxdg_output_v1_destroy(output->xdg_output);
wl_output_destroy(output->output);
destroy_buffer(&output->buffers[0]);
destroy_buffer(&output->buffers[1]);
@@ -283,6 +288,37 @@ const struct wl_seat_listener seat_listener = {
.name = seat_handle_name,
};
+static void add_layer_surface(struct swaybar_output *output) {
+ if (output->surface != NULL) {
+ return;
+ }
+ struct swaybar *bar = output->bar;
+
+ output->surface = wl_compositor_create_surface(bar->compositor);
+ assert(output->surface);
+ output->layer_surface = zwlr_layer_shell_v1_get_layer_surface(
+ bar->layer_shell, output->surface, output->output,
+ ZWLR_LAYER_SHELL_V1_LAYER_BOTTOM, "panel");
+ assert(output->layer_surface);
+ zwlr_layer_surface_v1_add_listener(output->layer_surface,
+ &layer_surface_listener, output);
+ zwlr_layer_surface_v1_set_anchor(output->layer_surface,
+ bar->config->position);
+}
+
+static bool bar_uses_output(struct swaybar *bar, const char *name) {
+ if (bar->config->all_outputs) {
+ return true;
+ }
+ struct config_output *coutput;
+ wl_list_for_each(coutput, &bar->config->outputs, link) {
+ if (strcmp(coutput->name, name) == 0) {
+ return true;
+ }
+ }
+ return false;
+}
+
static void output_geometry(void *data, struct wl_output *output, int32_t x,
int32_t y, int32_t width_mm, int32_t height_mm, int32_t subpixel,
const char *make, const char *model, int32_t transform) {
@@ -326,7 +362,22 @@ static void xdg_output_handle_logical_size(void *data,
static void xdg_output_handle_done(void *data,
struct zxdg_output_v1 *xdg_output) {
- // Who cares
+ struct swaybar_output *output = data;
+ struct swaybar *bar = output->bar;
+
+ assert(output->name != NULL);
+ if (!bar_uses_output(bar, output->name)) {
+ swaybar_output_free(output);
+ return;
+ }
+
+ if (wl_list_empty(&output->link)) {
+ wl_list_remove(&output->link);
+ wl_list_insert(&bar->outputs, &output->link);
+
+ add_layer_surface(output);
+ render_frame(bar, output);
+ }
}
static void xdg_output_handle_name(void *data,
@@ -349,17 +400,15 @@ struct zxdg_output_v1_listener xdg_output_listener = {
.description = xdg_output_handle_description,
};
-static bool bar_uses_output(struct swaybar *bar, const char *name) {
- if (bar->config->all_outputs) {
- return true;
- }
- struct config_output *coutput;
- wl_list_for_each(coutput, &bar->config->outputs, link) {
- if (strcmp(coutput->name, name) == 0) {
- return true;
- }
+static void add_xdg_output(struct swaybar_output *output) {
+ if (output->xdg_output != NULL) {
+ return;
}
- return false;
+ assert(output->bar->xdg_output_manager != NULL);
+ output->xdg_output = zxdg_output_manager_v1_get_xdg_output(
+ output->bar->xdg_output_manager, output->output);
+ zxdg_output_v1_add_listener(output->xdg_output, &xdg_output_listener,
+ output);
}
static void handle_global(void *data, struct wl_registry *registry,
@@ -386,7 +435,10 @@ static void handle_global(void *data, struct wl_registry *registry,
output->wl_name = name;
wl_list_init(&output->workspaces);
wl_list_init(&output->hotspots);
- wl_list_insert(&bar->outputs, &output->link);
+ wl_list_init(&output->link);
+ if (bar->xdg_output_manager != NULL) {
+ add_xdg_output(output);
+ }
} else if (strcmp(interface, zwlr_layer_shell_v1_interface.name) == 0) {
bar->layer_shell = wl_registry_bind(
registry, name, &zwlr_layer_shell_v1_interface, 1);
@@ -416,7 +468,9 @@ static const struct wl_registry_listener registry_listener = {
static void render_all_frames(struct swaybar *bar) {
struct swaybar_output *output;
wl_list_for_each(output, &bar->outputs, link) {
- render_frame(bar, output);
+ if (output->surface != NULL) {
+ render_frame(bar, output);
+ }
}
}
@@ -443,23 +497,10 @@ void bar_setup(struct swaybar *bar,
struct swaybar_output *output;
wl_list_for_each(output, &bar->outputs, link) {
- output->xdg_output = zxdg_output_manager_v1_get_xdg_output(
- bar->xdg_output_manager, output->output);
- zxdg_output_v1_add_listener(output->xdg_output, &xdg_output_listener,
- output);
+ add_xdg_output(output);
}
wl_display_roundtrip(bar->display);
- struct swaybar_output *output_tmp;
- wl_list_for_each_safe(output, output_tmp, &bar->outputs, link) {
- if (!bar_uses_output(bar, output->name)) {
- zxdg_output_v1_destroy(output->xdg_output);
- wl_output_destroy(output->output);
- wl_list_remove(&output->link);
- free(output);
- }
- }
-
struct swaybar_pointer *pointer = &bar->pointer;
int max_scale = 1;
@@ -479,18 +520,6 @@ void bar_setup(struct swaybar *bar,
pointer->cursor_surface = wl_compositor_create_surface(bar->compositor);
assert(pointer->cursor_surface);
- wl_list_for_each(output, &bar->outputs, link) {
- output->surface = wl_compositor_create_surface(bar->compositor);
- assert(output->surface);
- output->layer_surface = zwlr_layer_shell_v1_get_layer_surface(
- bar->layer_shell, output->surface, output->output,
- ZWLR_LAYER_SHELL_V1_LAYER_BOTTOM, "panel");
- assert(output->layer_surface);
- zwlr_layer_surface_v1_add_listener(output->layer_surface,
- &layer_surface_listener, output);
- zwlr_layer_surface_v1_set_anchor(output->layer_surface,
- bar->config->position);
- }
ipc_get_workspaces(bar);
render_all_frames(bar);
}
@@ -529,6 +558,7 @@ void bar_run(struct swaybar *bar) {
}
while (1) {
event_loop_poll();
+ wl_display_flush(bar->display);
}
}
diff --git a/swaybar/i3bar.c b/swaybar/i3bar.c
index 88404703..325aa61a 100644
--- a/swaybar/i3bar.c
+++ b/swaybar/i3bar.c
@@ -117,7 +117,9 @@ bool i3bar_handle_readable(struct status_line *status) {
memmove(status->buffer, &status->buffer[c], status->buffer_index);
break;
} else if (!isspace(status->buffer[c])) {
- status_error(status, "[invalid json]");
+ wlr_log(WLR_DEBUG, "Invalid i3bar json: expected '[' but encountered '%c'",
+ status->buffer[c]);
+ status_error(status, "[invalid i3bar json]");
return true;
}
}
@@ -155,6 +157,8 @@ bool i3bar_handle_readable(struct status_line *status) {
++buffer_pos;
break;
} else if (!isspace(status->buffer[buffer_pos])) {
+ wlr_log(WLR_DEBUG, "Invalid i3bar json: expected ',' but encountered '%c'",
+ status->buffer[buffer_pos]);
status_error(status, "[invalid i3bar json]");
return true;
}
@@ -166,7 +170,8 @@ bool i3bar_handle_readable(struct status_line *status) {
} else {
test_object = json_tokener_parse_ex(status->tokener,
&status->buffer[buffer_pos], status->buffer_index - buffer_pos);
- if (json_tokener_get_error(status->tokener) == json_tokener_success) {
+ enum json_tokener_error err = json_tokener_get_error(status->tokener);
+ if (err == json_tokener_success) {
if (json_object_get_type(test_object) == json_type_array) {
if (last_object) {
json_object_put(last_object);
@@ -198,12 +203,14 @@ bool i3bar_handle_readable(struct status_line *status) {
continue; // look for comma without reading more input
}
buffer_pos = status->buffer_index = 0;
- } else if (json_tokener_get_error(status->tokener) == json_tokener_continue) {
+ } else if (err == json_tokener_continue) {
+ json_tokener_reset(status->tokener);
if (status->buffer_index < status->buffer_size) {
// move the object to the start of the buffer
status->buffer_index -= buffer_pos;
memmove(status->buffer, &status->buffer[buffer_pos],
status->buffer_index);
+ buffer_pos = 0;
} else {
// expand buffer
status->buffer_size *= 2;
@@ -217,6 +224,10 @@ bool i3bar_handle_readable(struct status_line *status) {
}
}
} else {
+ char last_char = status->buffer[status->buffer_index - 1];
+ status->buffer[status->buffer_index - 1] = '\0';
+ wlr_log(WLR_DEBUG, "Failed to parse i3bar json - %s: '%s%c'",
+ json_tokener_error_desc(err), &status->buffer[buffer_pos], last_char);
status_error(status, "[failed to parse i3bar json]");
return true;
}
diff --git a/swaybar/render.c b/swaybar/render.c
index 97690338..26db80cb 100644
--- a/swaybar/render.c
+++ b/swaybar/render.c
@@ -1,4 +1,5 @@
#define _POSIX_C_SOURCE 200809L
+#include <assert.h>
#include <limits.h>
#include <stdlib.h>
#include <stdint.h>
@@ -480,6 +481,8 @@ static uint32_t render_to_cairo(cairo_t *cairo,
}
void render_frame(struct swaybar *bar, struct swaybar_output *output) {
+ assert(output->surface != NULL);
+
struct swaybar_hotspot *hotspot, *tmp;
wl_list_for_each_safe(hotspot, tmp, &output->hotspots, link) {
if (hotspot->destroy) {
@@ -507,7 +510,6 @@ void render_frame(struct swaybar *bar, struct swaybar_output *output) {
// TODO: this could infinite loop if the compositor assigns us a
// different height than what we asked for
wl_surface_commit(output->surface);
- wl_display_roundtrip(bar->display);
} else if (height > 0) {
// Replay recording into shm and send it off
output->current_buffer = get_next_buffer(bar->shm,
@@ -533,7 +535,6 @@ void render_frame(struct swaybar *bar, struct swaybar_output *output) {
wl_surface_damage(output->surface, 0, 0,
output->width, output->height);
wl_surface_commit(output->surface);
- wl_display_roundtrip(bar->display);
}
cairo_surface_destroy(recorder);
cairo_destroy(cairo);
generated by cgit v1.2.3 (git 2.39.1) at 2025-05-18 11:07:09 +0000