From 5ffa7f8f45fe3d0aed56a2ec299408546ee59b85 Mon Sep 17 00:00:00 2001
From: Qwerasd <qwerasd205@users.noreply.github.com>
Date: Tue, 18 Nov 2025 16:13:15 -0700
Subject: [PATCH] perf: inline calls to StaticBitSet.isSet in sgr parser

---
 src/terminal/sgr.zig | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/src/terminal/sgr.zig b/src/terminal/sgr.zig
index dc9505d14..6fd4f1e79 100644
--- a/src/terminal/sgr.zig
+++ b/src/terminal/sgr.zig
@@ -197,7 +197,12 @@ pub const Parser = struct {
         }
 
         const slice = self.params[self.idx..self.params.len];
-        const colon = self.params_sep.isSet(self.idx);
+        // Call inlined for performance reasons.
+        const colon = @call(
+            .always_inline,
+            SepList.isSet,
+            .{ self.params_sep, self.idx },
+        );
         self.idx += 1;
 
         // Our last one will have an idx be the last value.
@@ -485,10 +490,13 @@ pub const Parser = struct {
     /// Returns true if the present position has a colon separator.
     /// This always returns false for the last value since it has no
     /// separator.
-    fn isColon(self: *Parser) bool {
-        // The `- 1` here is because the last value has no separator.
-        if (self.idx >= self.params.len - 1) return false;
-        return self.params_sep.isSet(self.idx);
+    inline fn isColon(self: *Parser) bool {
+        // Call inlined for performance reasons.
+        return @call(
+            .always_inline,
+            SepList.isSet,
+            .{ self.params_sep, self.idx },
+        );
     }
 
     fn countColon(self: *Parser) usize {
@@ -514,7 +522,9 @@ fn testParse(params: []const u16) Attribute {
 }
 
 fn testParseColon(params: []const u16) Attribute {
-    var p: Parser = .{ .params = params, .params_sep = .initFull() };
+    var p: Parser = .{ .params = params };
+    // Mark all parameters except the last as having a colon after.
+    for (0..params.len - 1) |i| p.params_sep.set(i);
     return p.next().?;
 }