git.lucas.co / cce-compositor
Wayland compositor (wlroots)
git clone https://git.lucas.co/cce-compositor.git

commit8dfd6bc07acd345c07794ff0db462540baff696b
parentc46797eb0e
authorIsaac Freund <[email protected]>
date2024-01-04 14:44
Cursor: clean up cursor theme loading

wlroots will now load xcursor themes at the correct scale automatically
based on the scale of the outputs where ther cursors are displayed.

Also make the error handling a bit more robust.

 deps/zig-wlroots |  2 +-
 river/Cursor.zig | 26 +++++++++-----------------
 river/Output.zig |  9 ---------
 3 files changed, 10 insertions(+), 27 deletions(-)

diff --git a/deps/zig-wlroots b/deps/zig-wlroots
index 0ddfe81..68ce607 160000
--- a/deps/zig-wlroots
+++ b/deps/zig-wlroots
@@ -1 +1 @@
-Subproject commit 0ddfe81c5957fd36f8f8faf3d0870df974860661
+Subproject commit 68ce60769c6a8019c43de35197f4596eaac4df48
diff --git a/river/Cursor.zig b/river/Cursor.zig
index 99a0e3b..e74b300 100644
--- a/river/Cursor.zig
+++ b/river/Cursor.zig
@@ -237,20 +237,11 @@ pub fn deinit(self: *Self) void {
 pub fn setTheme(self: *Self, theme: ?[*:0]const u8, _size: ?u32) !void {
     const size = _size orelse default_size;
 
-    self.xcursor_manager.destroy();
-    self.xcursor_manager = try wlr.XcursorManager.create(theme, size);
-
-    // For each output, ensure a theme of the proper scale is loaded
-    var it = server.root.active_outputs.iterator(.forward);
-    while (it.next()) |output| {
-        const wlr_output = output.wlr_output;
-        self.xcursor_manager.load(wlr_output.scale) catch
-            log.err("failed to load xcursor theme '{?s}' at scale {}", .{ theme, wlr_output.scale });
-    }
+    const xcursor_manager = try wlr.XcursorManager.create(theme, size);
+    errdefer xcursor_manager.destroy();
 
     // If this cursor belongs to the default seat, set the xcursor environment
-    // variables as well as the xwayland cursor theme and update the cursor
-    // image if necessary.
+    // variables as well as the xwayland cursor theme.
     if (self.seat == server.input_manager.defaultSeat()) {
         const size_str = try std.fmt.allocPrintZ(util.gpa, "{}", .{size});
         defer util.gpa.free(size_str);
@@ -258,11 +249,8 @@ pub fn setTheme(self: *Self, theme: ?[*:0]const u8, _size: ?u32) !void {
         if (theme) |t| if (c.setenv("XCURSOR_THEME", t, 1) < 0) return error.OutOfMemory;
 
         if (build_options.xwayland) {
-            self.xcursor_manager.load(1) catch {
-                log.err("failed to load xcursor theme '{?s}' at scale 1", .{theme});
-                return;
-            };
-            const wlr_xcursor = self.xcursor_manager.getXcursor("left_ptr", 1).?;
+            try xcursor_manager.load(1);
+            const wlr_xcursor = xcursor_manager.getXcursor("left_ptr", 1).?;
             const image = wlr_xcursor.images[0];
             server.xwayland.setCursor(
                 image.buffer,
@@ -275,6 +263,10 @@ pub fn setTheme(self: *Self, theme: ?[*:0]const u8, _size: ?u32) !void {
         }
     }
 
+    // Everything fallible is now done so the the old xcursor_manager can be destroyed.
+    self.xcursor_manager.destroy();
+    self.xcursor_manager = xcursor_manager;
+
     if (self.xcursor_name) |name| {
         self.setXcursor(name);
     }
diff --git a/river/Output.zig b/river/Output.zig
index 04d92ff..1c092ee 100644
--- a/river/Output.zig
+++ b/river/Output.zig
@@ -306,15 +306,6 @@ pub fn create(wlr_output: *wlr.Output) !void {
     wlr_output.events.frame.add(&output.frame);
     wlr_output.events.present.add(&output.present);
 
-    // Ensure that a cursor image at the output's scale factor is loaded
-    // for each seat.
-    var it = server.input_manager.seats.first;
-    while (it) |seat_node| : (it = seat_node.next) {
-        const seat = &seat_node.data;
-        seat.cursor.xcursor_manager.load(wlr_output.scale) catch
-            log.err("failed to load xcursor theme at scale {}", .{wlr_output.scale});
-    }
-
     output.setTitle();
 
     output.active_link.init();