A refused Delcol changes nothing: no +Unsaved opens, no focus moves

Delcol's refusal opened +Unsaved, placing a pane in the layout it was refusing to change. Its unsaved records and notice say which panes; the list pane is left to the refusals of the other words.

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

1 files changed, 6 insertions, 3 deletions

diff --git a/src/builtins.zig b/src/builtins.zig
index 4a0b107c..e47afeb4 100644
--- a/src/builtins.zig
+++ b/src/builtins.zig
@@ -562,7 +562,9 @@ fn warnModifiedIn(c: Ctx, asking: Pane.Discarding, which: Asked) bool {
     }
     if (count > 0) {
         listed.print(arena, "\n{s} again to discard\n", .{@tagName(asking)}) catch {};
-        if (c.p.gpa.dupe(u8, listed.items)) |content| panes.Output.openUnsaved(c.p, c.id, content) else |_| {}
+        // A refused Delcol changes nothing, the layout included: its records
+        // and its notice say it, and no +Unsaved opens.
+        if (asking != .Delcol) if (c.p.gpa.dupe(u8, listed.items)) |content| panes.Output.openUnsaved(c.p, c.id, content) else |_| {};
         var short: [96]u8 = undefined;
         c.p.reportFailure(c.id, std.fmt.bufPrint(&short, "{d} unsaved pane{s} \u{2014} {s} again to discard", .{ count, if (count == 1) "" else "s", @tagName(asking) }) catch "unsaved panes");
     }
@@ -2003,7 +2005,7 @@ test "Exit and Restore name a modified +New scratch as well as files" {
     // Delcol, Restore and Exit name it the same way.
     p.fs.failure_len = 0;
     try std.testing.expect(p.executeBuiltinLine(p.active, "Delcol"));
-    try std.testing.expect(std.mem.indexOf(u8, unsavedList(p), "Delcol again to discard") != null);
+    try std.testing.expect(std.mem.indexOf(u8, unsavedList(p), "Delcol again to discard") == null);
     try std.testing.expect(std.mem.indexOf(u8, unsavedList(p), named[0 .. config.scratch_buffer.len + 7]) != null);
 }
 
@@ -2016,7 +2018,8 @@ test "Del and Delcol refuse a pane with unsaved text once, as acme's do, and clo
     const serial = pane.serial;
     try std.testing.expect(p.executeBuiltinLine(p.active, "Delcol"));
     try std.testing.expect(p.paneBySerial(serial) != null);
-    try std.testing.expect(std.mem.endsWith(u8, unsavedList(p), ": Modified\n\nDelcol again to discard\n"));
+    // A refused Delcol changes nothing: no +Unsaved (its records say it).
+    try std.testing.expect(std.mem.indexOf(u8, unsavedList(p), "Delcol") == null);
     // Each word is warned on its own: Del asks too, then closes.
     try std.testing.expect(p.executeBuiltinLine(p.active, "Del"));
     try std.testing.expect(p.paneBySerial(serial) != null);