From: Gerd Moellmann Date: Tue, 14 Nov 2000 04:41:56 +0000 (+0000) Subject: (byte-compile-unfold-lambda): Don't X-Git-Tag: emacs-pretest-21.0.90~89 X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=936ae731f35a288ebd5fc875f4123eaa286c5f50;p=emacs.git (byte-compile-unfold-lambda): Don't recursively optimize body because that can lead to infinite recursion; see comment there. --- diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 843de781e06..50fd8376e45 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,3 +1,9 @@ +2000-11-14 Gerd Moellmann + + * emacs-lisp/byte-opt.el (byte-compile-unfold-lambda): Don't + recursively optimize body because that can lead to infinite + recursion; see comment there. + 2000-11-13 Eli Zaretskii * faces.el (face-spec-set-match-display): Revert the change from diff --git a/lisp/emacs-lisp/byte-opt.el b/lisp/emacs-lisp/byte-opt.el index c3c8e677239..da695386604 100644 --- a/lisp/emacs-lisp/byte-opt.el +++ b/lisp/emacs-lisp/byte-opt.el @@ -340,7 +340,12 @@ (byte-compile-warn "Attempt to open-code `%s' with too many arguments" name)) form) - (setq body (mapcar 'byte-optimize-form body)) + + ;; The following leads to infinite recursion when loading a + ;; file containing `(defsubst f () (f))', and then trying to + ;; byte-compile that file. + ;(setq body (mapcar 'byte-optimize-form body))) + (let ((newform (if bindings (cons 'let (cons (nreverse bindings) body))