style(imgui): use orelse instead of if () ||

lib/SDL.zig

@@ -1 +1 @@
-Subproject commit 59c90e8ea2135384410817ccfdcc15ce5f7a245e
+Subproject commit aa063fab4031d12213dd95f1db5bec9d4f6727bc

lib/known-folders

@@ -1 +1 @@
-Subproject commit 53fe3b676f32e59d46f4fd201d7ab200e5f6cb98
+Subproject commit d13ba6137084e55f873f6afb67447fe8906cc951

lib/zig-clap

@@ -1 +1 @@
-Subproject commit 861de651f3e1314973b1273ac7856e96b2625ff3
+Subproject commit cb13519431b916c05c6c783cb0ce3b232be5e400

lib/zig-datetime

@@ -1 +1 @@
-Subproject commit bf0ae0c27cfe92fdd9a92c8f1ac6d1939ae60c77
+Subproject commit b570d61187a64a32067720aa0c2385061685428a

src/imgui.zig

@@ -21,8 +21,8 @@ const gba_height = @import("core/ppu.zig").height;
 
 const log = std.log.scoped(.Imgui);
 
-// TODO: Document how I decided on this value (I forgot 😅)
-const histogram_len = 0x400;
+// two seconds worth of fps values into the past
+const histogram_len = 0x80;
 
 /// Immediate-Mode GUI State
 pub const State = struct {
@@ -37,10 +37,7 @@ pub const State = struct {
         var title: [12:0]u8 = [_:0]u8{0} ** 12;
         std.mem.copy(u8, &title, "[No Title]");
 
-        return .{
-            .title = title,
-            .fps_hist = RingBuffer(u32).init(history),
-        };
+        return .{ .title = title, .fps_hist = RingBuffer(u32).init(history) };
     }
 
     pub fn deinit(self: *@This(), allocator: Allocator) void {
@@ -67,17 +64,19 @@ pub fn draw(state: *State, tex_id: GLuint, cpu: *Arm7tdmi) void {
                     break :blk;
                 };
 
-                if (maybe_path) |file_path| {
-                    defer nfd.freePath(file_path);
-                    log.info("user chose: \"{s}\"", .{file_path});
+                const file_path = maybe_path orelse {
+                    log.warn("did not receive a file path", .{});
+                    break :blk;
+                };
+                defer nfd.freePath(file_path);
 
-                    emu.replaceGamepak(cpu, file_path) catch |e| {
-                        log.err("failed to replace GamePak: {}", .{e});
-                        break :blk;
-                    };
+                log.info("user chose: \"{s}\"", .{file_path});
+                emu.replaceGamepak(cpu, file_path) catch |e| {
+                    log.err("failed to replace GamePak: {}", .{e});
+                    break :blk;
+                };
 
-                    state.title = cpu.bus.pak.title ++ [_:0]u8{};
-                }
+                state.title = cpu.bus.pak.title ++ [_:0]u8{};
             }
         }
 
@@ -130,7 +129,7 @@ pub fn draw(state: *State, tex_id: GLuint, cpu: *Arm7tdmi) void {
         defer zgui.end();
 
         const tmp = blk: {
-            var buf: [0x400]u32 = undefined;
+            var buf: [histogram_len]u32 = undefined;
             const len = state.fps_hist.copy(&buf);
 
             break :blk .{ buf, len };