Ensure undo-boundary after all commands

* lisp/simple.el (undo-auto--boundaries): Ensure an undo-boundary after
  every command whether it (apparently) changes the buffer or not.

See Bug#23785 for discussion.

diff --git a/lisp/simple.el b/lisp/simple.el
index a502448b31cb2525dd489b518860893380cbe1a8..bc3e7b83b8d8b2e8a3fd4debc63d150340894562 100644 (file)
--- a/lisp/simple.el
+++ b/lisp/simple.el
@@ -2884,6 +2884,10 @@ REASON describes the reason that the boundary is being added; see
   "Check recently changed buffers and add a boundary if necessary.
 REASON describes the reason that the boundary is being added; see
 `undo-last-boundary' for more information."
+  ;; (Bug #23785) All commands should ensure that there is an undo
+  ;; boundary whether they have changed the current buffer or not.
+  (when (eq cause 'command)
+    (add-to-list 'undo-auto--undoably-changed-buffers (current-buffer)))
   (dolist (b undo-auto--undoably-changed-buffers)
           (when (buffer-live-p b)
             (with-current-buffer b