Move exec out of pardes.zig into exec.zig

Pure move, no behaviour change (acme keeps this in exec.c): execute,
commandText, max_exec_depth, executeBuiltinLine, applyStartupConfig,
runBuiltin and applySettingBuiltin; the getarg-style operand code
(withArg, PointerOperand, pointerOperand, heldSelection, chordEachSel);
takesCommandLine and the terminals commands run in (spawnTty, spawnV9fsTty,
spawnTtyWithMount, evictLonePristineTty, replaceStillborn, ttyForDir);
placeDoc; and the save path (submitSave, saveFile, saveTo, askWrite), with
the acme-chords test, go verbatim to exec.zig.

The methods become free functions taking `p: *Pardes`. executeBuiltinLine
is called from ~170 places as `p.executeBuiltinLine(..)`, so Pardes keeps
one declaration alias for it and those call sites stay; the other 104 calls
change from `p.execute(..)` to `exec.execute(p, ..)`. ninep/ctl.zig has a
local named exec, so it writes `pardes.exec.execute` instead of importing
the file.

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

1 files changed, 2 insertions, 1 deletions

diff --git a/src/pdf_view.zig b/src/pdf_view.zig
index 9b623183..721dac5a 100644
--- a/src/pdf_view.zig
+++ b/src/pdf_view.zig
@@ -3,6 +3,7 @@
 const panes = @import("panes.zig");
 const std = @import("std");
 const pardes = @import("pardes.zig");
+const exec = @import("exec.zig");
 const layout = @import("layout.zig");
 const config = @import("config.zig");
 const modal = @import("modal.zig");
@@ -2044,7 +2045,7 @@ pub fn openSections(core: *pardes.Pardes, id: usize) void {
         .serial = result.serial,
         .revision = result.file.?.revision,
     };
-    core.placeDoc(id, free, result);
+    exec.placeDoc(core, id, free, result);
     layout.compute(core);
     core.active = id;
     pane.search_pane = free;