A look that goes to another pane leaves the looking pane's dot alone

Every look that found something cleared the looker's selection first, so
`@p5:2` or `f.txt:2` written to a pane's look collapsed its dot to
q1-1. Only a look that stays in the pane starts from a clean selection
now; one that goes elsewhere takes away just the click's highlight, and a
miss changes nothing, as acme's look does.

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

1 files changed, 24 insertions, 0 deletions

diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index 709f017f..b8f73c6f 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -2024,6 +2024,30 @@ test "@p<serial>:<address> takes any sam address; a miss and an unknown serial a
     try testing.expectEqualStrings("", rd(p, root, 0, 64).bytes);
 }
 
+test "a look that goes to another pane leaves the looking pane's dot alone" {
+    const p = try withFile(testing.allocator, "alpha bravo\ncharlie\n");
+    defer p.deinit();
+    const source = serialOf(p);
+    const other = try th.newPane(p);
+    _ = wr(p, Node.of(other, .body), "one\ntwo\nthree\n");
+    var tmp = testing.tmpDir(.{});
+    defer tmp.cleanup();
+    try tmp.dir.writeFile(testing.io, .{ .sub_path = "f.txt", .data = "f one\nf two\n" });
+    var dir_buf: [4096]u8 = undefined;
+    const dir = dir_buf[0..try tmp.dir.realPath(testing.io, &dir_buf)];
+    var line: [4200]u8 = undefined;
+    for ([_][]const u8{ "@p{d}:2", "@p{d}:/three/", "{s}/f.txt:2" }) |form| {
+        @import("pane.zig").setDot(p.panes[p.paneBySerial(source).?].?, .{ .q0 = 6, .q1 = 11 });
+        const text = if (form[0] == '@')
+            (if (std.mem.eql(u8, form, "@p{d}:2")) try std.fmt.bufPrint(&line, "@p{d}:2\n", .{other}) else try std.fmt.bufPrint(&line, "@p{d}:/three/\n", .{other}))
+        else
+            try std.fmt.bufPrint(&line, "{s}/f.txt:2\n", .{dir});
+        try testing.expectEqual(Status.ok, wr(p, Node.of(source, .look), text).reply.status);
+        try testing.expect(p.active != p.paneBySerial(source).?);
+        try testing.expectEqualStrings("          6          11 ", rd(p, Node.of(source, .dot), 0, 64).bytes);
+    }
+}
+
 test "@p<terminal>:<address> takes any address over the terminal's lines, selecting the match" {
     const p = try th.withTerm(testing.allocator);
     defer p.deinit();