From 927cb4e6225f6fbe9a7de4b7a0c9aa0e68e26481 Mon Sep 17 00:00:00 2001
From: Mitchell Hashimoto <m@mitchellh.com>
Date: Tue, 11 Feb 2025 16:52:25 -0800
Subject: [PATCH] apprt/gtk: some stylistic changes

---
 src/apprt/gtk/App.zig | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/apprt/gtk/App.zig b/src/apprt/gtk/App.zig
index 41bc755c2..f9a3ab160 100644
--- a/src/apprt/gtk/App.zig
+++ b/src/apprt/gtk/App.zig
@@ -560,9 +560,14 @@ pub fn performAction(
         .render_inspector,
         .renderer_health,
         .color_change,
-        => log.warn("unimplemented action={}", .{action}),
+        => {
+            log.warn("unimplemented action={}", .{action});
+            return false;
+        },
     }
 
+    // We can assume it was handled because all unknown/unimplemented actions
+    // are caught above.
     return true;
 }
 
@@ -602,9 +607,7 @@ fn closeTab(_: *App, target: apprt.Target) !void {
 
 fn gotoTab(_: *App, target: apprt.Target, tab: apprt.action.GotoTab) bool {
     switch (target) {
-        .app => {
-            return false;
-        },
+        .app => return false,
         .surface => |v| {
             const window = v.rt_surface.container.window() orelse {
                 log.info(
@@ -675,9 +678,7 @@ fn gotoSplit(
     direction: apprt.action.GotoSplit,
 ) bool {
     switch (target) {
-        .app => {
-            return false;
-        },
+        .app => return false,
         .surface => |v| {
             const s = v.rt_surface.container.split() orelse return false;
             const map = s.directionMap(switch (v.rt_surface.container) {