The PDF benches share their clock, fatal exit, hash, count parser and effect drain, and lspprobe is lspbench's probe subcommand

B4: test/bench_util.zig holds what pdf_bench, pdf_pointer_bench, pdf_scroll_bench and pdf_sections_bench each copied (nowNs x4, fatal x3 named per bench, mix x3, parsePositive/parseCount x3, drainEffects x2). They stay four entry points: pdf_bench links its own mupdf module (with its C source and c_heap) where the others link the core's, which one executable cannot import under one name, and their reports and identities are each bench's own. pdf_sections_bench had rotted since the pane's cursor and selections moved to its body and the render revision became next_raster_revision; it builds and runs again. C6: tools/lspprobe.zig is lspbench's `probe` subcommand (zig build lspbench -- probe <kind> <file> <line>:<col>), and the lspprobe build step goes; docs/lsp.md names the new command.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 5 insertions, 18 deletions

diff --git a/test/pdf_bench.zig b/test/pdf_bench.zig
index 0f09f72a..23565106 100644
--- a/test/pdf_bench.zig
+++ b/test/pdf_bench.zig
@@ -13,6 +13,9 @@
 //! filesystem cache is deliberately uncontrolled), while warm scenarios
 //! retain one Document.
 const std = @import("std");
+const bench_util = @import("bench_util.zig");
+const nowNs = bench_util.nowNs;
+const fatal = bench_util.Fatal("pdf-bench").f;
 const pdf = @import("mupdf");
 
 pub const std_options: std.Options = .{ .log_level = .err };
@@ -205,14 +208,6 @@ fn parseArgs(args: []const []const u8) Config {
     return config;
 }
 
-fn parsePositive(name: []const u8, text: []const u8, maximum: usize) usize {
-    const value = std.fmt.parseUnsigned(usize, text, 10) catch
-        fatal("{s} expects a positive integer", .{name});
-    if (value == 0 or value > maximum)
-        fatal("{s} must be in 1..{d}", .{ name, maximum });
-    return value;
-}
-
 fn usage() noreturn {
     printUsage();
     std.process.exit(2);
@@ -226,13 +221,6 @@ fn printUsage() void {
     );
 }
 
-fn nowNs() u64 {
-    var ts: std.c.timespec = undefined;
-    _ = std.c.clock_gettime(.MONOTONIC, &ts);
-    return @as(u64, @intCast(ts.sec)) *| 1_000_000_000 +|
-        @as(u64, @intCast(ts.nsec));
-}
-
 fn measure(
     config: Config,
     profile_index: usize,
@@ -617,7 +605,6 @@ fn reportJson(
     std.Io.File.stdout().writeStreamingAll(io, out.buffered()) catch {};
 }
 
-fn fatal(comptime format: []const u8, args: anytype) noreturn {
-    std.debug.print("pdf-bench: " ++ format ++ "\n", args);
-    std.process.exit(1);
+fn parsePositive(name: []const u8, text: []const u8, maximum: usize) usize {
+    return bench_util.parsePositive(fatal, name, text, maximum);
 }