fix(tools): alloc adjusts

Author: kassane

.github/workflows/ci.yml

@@ -61,13 +61,13 @@ jobs:
         run: zig build sdk-build --summary all
 
       - name: Build NES examples and generate Mesen label files
-        run: zig build nes-hello1 nes-hello2 nes-hello3 nes-zig-logo nes-fade nes-sprites nes-random nes-pads nes-color-cycle nes-fullbg nes-mappers nes-cnrom-hello nes-unrom-hello nes-unrom-512-hello nes-mmc1-hello nes-mmc3-hello nes-gtrom-hello nes-action53-hello fds-hello gen-labels --summary all
+        run: zig build nes-hello1 nes-hello2 nes-hello3 nes-zig-logo nes-fade nes-sprites nes-random nes-pads nes-color-cycle nes-fullbg nes-mappers nes-bat-ball nes-megablast nes-gg-demo nes-cnrom-hello nes-cnrom-sprites nes-unrom-hello nes-unrom-512-hello nes-unrom-color-cycle nes-mmc1-hello nes-mmc1-sprites nes-mmc3-hello nes-mmc3-pads nes-gtrom-hello nes-gtrom-color-cycle nes-action53-hello fds-hello gen-labels --summary all
 
       - name: Build Commodore examples (C64, VIC-20, CX16, GEOS)
         run: zig build c64-hello c64-fibonacci c64-plasma vic20-hello cx16-hello cx16-k-console-test geos-hello --summary all
 
-      - name: Build Atari examples (Lynx, 2600, 8-bit)
-        run: zig build lynx-hello atari2600-colorbar atari2600-3e-colorbar atari8dos-hello atari8-cart-hello --summary all
+      - name: Build Atari examples (Lynx, 2600, 8-bit, 5200)
+        run: zig build lynx-hello atari2600-colorbar atari2600-3e-colorbar atari8dos-hello atari8-cart-hello atari8-megacart-hello atari8-xegs-hello atari5200-cart-hello --summary all
 
       - name: Build PC Engine, MEGA65, Neo6502 examples
         run: zig build pce-color-cycle pce-color-cycle-banked mega65-hello mega65-plasma mega65-viciv neo6502-graphics --summary all
@@ -84,15 +84,20 @@ jobs:
           ls -la zig-out/bin/
           for f in \
             hello1.nes hello2.nes hello3.nes zig-logo.nes fade.nes sprites.nes random.nes pads.nes color-cycle.nes \
-            fullbg.nes mappers.nes cnrom-hello.nes unrom-hello.nes unrom-512-hello.nes mmc1-hello.nes mmc3-hello.nes gtrom-hello.nes \
+            fullbg.nes mappers.nes bat-ball.nes megablast.nes gg-demo.nes \
+            cnrom-hello.nes cnrom-sprites.nes \
+            unrom-hello.nes unrom-512-hello.nes unrom-color-cycle.nes \
+            mmc1-hello.nes mmc1-sprites.nes \
+            mmc3-hello.nes mmc3-pads.nes \
+            gtrom-hello.nes gtrom-color-cycle.nes \
             action53-hello.nes fds-hello.fds \
             c64-hello.prg fibonacci.prg plasma.prg \
             vic20-hello.prg \
             cx16-hello.prg cx16-k-console-test.prg \
             geos-hello.cvt \
             lynx-hello.bll \
             colorbar.a26 colorbar-3e.a26 \
-            atari8dos-hello.xex atari8-cart-hello.rom \
+            atari8dos-hello.xex atari8-cart-hello.rom atari8-megacart-hello.rom atari8-xegs-hello.rom atari5200-cart-hello.rom \
             pce-color-cycle.pce pce-color-cycle-banked.pce \
             mega65-hello.prg viciv.prg \
             graphics.neo \
@@ -146,11 +151,15 @@ jobs:
           }
           # NES
           for f in hello1.nes hello2.nes hello3.nes zig-logo.nes fade.nes sprites.nes random.nes \
-                   pads.nes color-cycle.nes fullbg.nes mappers.nes cnrom-hello.nes unrom-hello.nes unrom-512-hello.nes mmc1-hello.nes mmc3-hello.nes; do
+                   pads.nes color-cycle.nes fullbg.nes mappers.nes bat-ball.nes megablast.nes gg-demo.nes \
+                   cnrom-hello.nes cnrom-sprites.nes \
+                   unrom-hello.nes unrom-512-hello.nes unrom-color-cycle.nes \
+                   mmc1-hello.nes mmc1-sprites.nes \
+                   mmc3-hello.nes mmc3-pads.nes; do
             smoke "$f"
           done
-          # Note: gtrom-hello.nes (mapper 111) is NOT tested here — apt mednafen lacks GTROM support.
-          # GTROM is covered by the emutest job (Mesen-X libretro core).
+          # Note: gtrom-hello.nes / gtrom-color-cycle.nes (mapper 111) are NOT tested here —
+          # apt mednafen lacks GTROM support. Both are covered by the emutest job (Mesen-X libretro core).
           # PC Engine
           for f in pce-color-cycle.pce pce-color-cycle-banked.pce; do
             smoke "$f"
@@ -245,8 +254,12 @@ jobs:
             echo "OK: $f (rc=$rc)"
           }
           for f in hello1.nes hello2.nes hello3.nes zig-logo.nes fade.nes sprites.nes random.nes \
-                   pads.nes color-cycle.nes fullbg.nes mappers.nes cnrom-hello.nes unrom-hello.nes unrom-512-hello.nes mmc1-hello.nes mmc3-hello.nes \
-                   gtrom-hello.nes; do
+                   pads.nes color-cycle.nes fullbg.nes mappers.nes bat-ball.nes megablast.nes gg-demo.nes \
+                   cnrom-hello.nes cnrom-sprites.nes \
+                   unrom-hello.nes unrom-512-hello.nes unrom-color-cycle.nes \
+                   mmc1-hello.nes mmc1-sprites.nes \
+                   mmc3-hello.nes mmc3-pads.nes \
+                   gtrom-hello.nes gtrom-color-cycle.nes; do
             run_emutest "$f" mesen_libretro.so
           done
           for f in colorbar.a26 colorbar-3e.a26; do

tools/bininfo.zig

@@ -151,12 +151,12 @@ fn dwarfLangStr(lang: u32) []const u8 {
 
 fn dumpDebugInfo(
     out: anytype,
+    gpa: std.mem.Allocator,
     debug_info: []const u8,
     debug_abbrev: []const u8,
     debug_str: []const u8,
     debug_ranges: []const u8,
 ) void {
-    const gpa = std.heap.page_allocator;
     const Dw = std.debug.Dwarf;
     const Id = Dw.Section.Id;
 
@@ -276,7 +276,7 @@ fn dumpDebugLine(out: anytype, debug_line: []const u8) void {
 
 // ── ELF inspector ─────────────────────────────────────────────────────────────
 
-fn checkElf(out: anytype, path: []const u8, data: []const u8, opts: Opts) bool {
+fn checkElf(out: anytype, gpa: std.mem.Allocator, path: []const u8, data: []const u8, opts: Opts) bool {
     if (data.len < 52) {
         out.print("{s}: [ELF] ERROR: header truncated ({d} B)\n", .{ path, data.len }) catch {};
         return false;
@@ -583,7 +583,7 @@ fn checkElf(out: anytype, path: []const u8, data: []const u8, opts: Opts) bool {
                     data[rng_off..][0..rng_sz]
                 else
                     &[_]u8{};
-                dumpDebugInfo(out, data[di_off..][0..di_sz], data[ab_off..][0..ab_sz], str_data, rng_data);
+                dumpDebugInfo(out, gpa, data[di_off..][0..di_sz], data[ab_off..][0..ab_sz], str_data, rng_data);
             }
 
             // Parse .debug_line file tables.
@@ -1224,10 +1224,10 @@ fn disasm6502(out: anytype, data: []const u8, load_addr: u16) void {
 
 // ── dispatch ──────────────────────────────────────────────────────────────────
 
-fn checkFile(out: anytype, path: []const u8, data: []const u8, opts: Opts) bool {
+fn checkFile(out: anytype, gpa: std.mem.Allocator, path: []const u8, data: []const u8, opts: Opts) bool {
     // Detect by magic first.
     if (data.len >= 4 and std.mem.eql(u8, data[0..4], "\x7FELF"))
-        return checkElf(out, path, data, opts);
+        return checkElf(out, gpa, path, data, opts);
     if (data.len >= 4 and std.mem.eql(u8, data[0..4], "NES\x1a"))
         return checkNes(out, path, data);
     if (data.len >= 4 and std.mem.eql(u8, data[0..4], "FDS\x1a"))
@@ -1395,7 +1395,7 @@ pub fn main(init: std.process.Init) !void {
             };
             defer alloc.free(data);
 
-            if (!checkFile(stdout, arg, data, opts)) all_ok = false;
+            if (!checkFile(stdout, alloc, arg, data, opts)) all_ok = false;
 
             if (opts.xxd) {
                 stdout.print("  -- xxd " ++ "-" ** 47 ++ "\n", .{}) catch {};

tools/elf2mlb.zig

@@ -113,7 +113,7 @@ pub fn main(init: std.process.Init) !void {
     const sym_count = symtab_size / @sizeOf(elf.Elf32_Sym);
 
     // Build MLb content in an in-memory buffer.
-    var mlb_buf: std.ArrayList(u8) = .empty;
+    var mlb_buf: std.ArrayListUnmanaged(u8) = .empty;
     defer mlb_buf.deinit(alloc);
 
     var count: usize = 0;