diff --git a/src/main.zig b/src/main.zig index c10c3b4b..e697d0c6 100644 --- a/src/main.zig +++ b/src/main.zig @@ -31,12 +31,13 @@ pub fn main() !void { // allocator // - in Debug mode we use the General Purpose Allocator to detect memory leaks // - in Release mode we use the c allocator - var gpa_instance: std.heap.DebugAllocator(.{ .stack_trace_frames = 10 }) = .init; - const gpa = if (builtin.mode == .Debug) gpa_instance.allocator() else std.heap.c_allocator; + // var gpa_instance: std.heap.DebugAllocator(.{ .stack_trace_frames = 10 }) = .init; + // const gpa = if (builtin.mode == .Debug) gpa_instance.allocator() else std.heap.c_allocator; + const gpa = std.heap.c_allocator; - defer if (builtin.mode == .Debug) { - if (gpa_instance.detectLeaks()) std.posix.exit(1); - }; + // defer if (builtin.mode == .Debug) { + // if (gpa_instance.detectLeaks()) std.posix.exit(1); + // }; // arena for main-specific allocations var main_arena_instance = std.heap.ArenaAllocator.init(gpa); diff --git a/src/telemetry/lightpanda.zig b/src/telemetry/lightpanda.zig index 4517d74b..e9553f27 100644 --- a/src/telemetry/lightpanda.zig +++ b/src/telemetry/lightpanda.zig @@ -28,7 +28,7 @@ run_mode: Config.RunMode = .serve, head: std.atomic.Value(usize) = .init(0), tail: std.atomic.Value(usize) = .init(0), -dropped: std.atomic.Value(usize) = .init(0), +dropped: std.atomic.Value(u32) = .init(0), buffer: [BUFFER_SIZE]telemetry.Event = undefined, pub fn init(self: *LightPanda, app: *App, iid: ?[36]u8, run_mode: Config.RunMode) !void { diff --git a/src/telemetry/telemetry.zig b/src/telemetry/telemetry.zig index e79003f3..c79408e5 100644 --- a/src/telemetry/telemetry.zig +++ b/src/telemetry/telemetry.zig @@ -103,16 +103,15 @@ pub const Event = union(enum) { run: void, navigate: Navigate, buffer_overflow: BufferOverflow, - flag: []const u8, // used for testing const Navigate = struct { tls: bool, proxy: bool, - driver: []const u8 = "cdp", + driver: enum { cdp } = .cdp, }; const BufferOverflow = struct { - dropped: usize, + dropped: u32, }; }; @@ -166,13 +165,13 @@ test "telemetry: sends event to provider" { telemetry.disabled = false; const mock = telemetry.provider; - telemetry.record(.{ .flag = "1" }); - telemetry.record(.{ .flag = "2" }); - telemetry.record(.{ .flag = "3" }); + telemetry.record(.{ .buffer_overflow = .{ .dropped = 1 } }); + telemetry.record(.{ .buffer_overflow = .{ .dropped = 2 } }); + telemetry.record(.{ .buffer_overflow = .{ .dropped = 3 } }); try testing.expectEqual(3, mock.events.items.len); for (mock.events.items, 0..) |event, i| { - try testing.expectEqual(i + 1, std.fmt.parseInt(usize, event.flag, 10)); + try testing.expectEqual(i + 1, event.buffer_overflow.dropped); } }