fix build on Zig 0.16.0-dev.728+87c18945c

[ataias]

I bumped Zig to 0.16.0-dev.728+87c18945c but zls failed to build.

[llogick]

Failures are due to ArrayList pop() now setting the previously last element to undefined (ziglang/zig@87c1894), which affects the value of 'curr_ctx' in Analysis.getPositionContext()

quickest way around it

diff --git a/src/analysis.zig b/src/analysis.zig
index e4a1ea22..07558c1a 100644
--- a/src/analysis.zig
+++ b/src/analysis.zig
@@ -5035,13 +5035,15 @@ pub fn getPositionContext(
             },
             .l_bracket => try stack.append(allocator, .{ .ctx = .empty, .stack_id = .bracket }),
             .r_paren => {
-                _ = stack.pop();
+                // Do this manually, as .pop() sets `stack.items[stack.items.len - 1]` to `undefined` which currently curr_ctx points to
+                if (stack.items.len != 0) stack.items.len -= 1;
                 if (curr_ctx.stack_id != .paren) {
                     (try peek(allocator, &stack)).ctx = .empty;
                 }
             },
             .r_bracket => {
-                _ = stack.pop();
+                // Do this manually, as .pop() sets `stack.items[stack.items.len - 1]` to `undefined` which currently curr_ctx points to
+                if (stack.items.len != 0) stack.items.len -= 1;
                 if (curr_ctx.stack_id != .bracket) {
                     (try peek(allocator, &stack)).ctx = .empty;
                 }

[ataias]

@llogick Thanks! I've applied your suggestion and this time I double checked that the zig build check test step passes locally.

[Techatrix]

The minimum zig version needs to be updated in the build.zig and build.zig.zon as well as the zig version in the nix flake. I will take care of that.