> While looking at this bug report, I found another bug. The menu and > toolbar item "undo" doesn't become disabled when the pending undo > list is empty.
I found another place where the predicate `listp' is missing for `pending-undo-list'. Test case: M-: a b c C-a C-t displays the error message: undo-more: No further undo information which has nothing to do with transposing two characters. It correctly fails because it can't transpose read-only characters, but its error message is wrong. This could be fixed by the patch below. It displays the correct error message: transpose-subr-1: Text is read-only Index: lisp/subr.el =================================================================== RCS file: /sources/emacs/emacs/lisp/subr.el,v retrieving revision 1.491 diff -c -r1.491 subr.el *** lisp/subr.el 28 Dec 2005 16:36:18 -0000 1.491 --- lisp/subr.el 29 Dec 2005 01:01:55 -0000 *************** *** 1686,1692 **** (when (and (consp elt) (not (eq elt (last pending-undo-list)))) (error "Undoing to some unrelated state")) ;; Undo it all. ! (while pending-undo-list (undo-more 1)) ;; Reset the modified cons cell ELT to its original content. (when (consp elt) (setcar elt old-car) --- 1686,1692 ---- (when (and (consp elt) (not (eq elt (last pending-undo-list)))) (error "Undoing to some unrelated state")) ;; Undo it all. ! (while (listp pending-undo-list) (undo-more 1)) ;; Reset the modified cons cell ELT to its original content. (when (consp elt) (setcar elt old-car) -- Juri Linkov http://www.jurta.org/emacs/ _______________________________________________ emacs-pretest-bug mailing list emacs-pretest-bug@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug