]> git.eshelyaron.com Git - emacs.git/commitdiff
cl-generic.el: Fix bug#46722
authorStefan Monnier <monnier@iro.umontreal.ca>
Wed, 5 Jan 2022 19:28:08 +0000 (14:28 -0500)
committerStefan Monnier <monnier@iro.umontreal.ca>
Wed, 5 Jan 2022 19:28:37 +0000 (14:28 -0500)
Fix longstanding bug due to unexpected interference via side-effect.

* lisp/emacs-lisp/cl-generic.el (cl--generic-get-dispatcher):
Copy the `dispatch` arg before storing it into the hash-table.

lisp/emacs-lisp/cl-generic.el

index 983694204df52b73a57468a30833e1a8c40774ea..5090e060372c42e90262abbd7ffb362a230cc12e 100644 (file)
@@ -604,7 +604,9 @@ The set of acceptable TYPEs (also called \"specializers\") is defined
 
 (defun cl--generic-get-dispatcher (dispatch)
   (with-memoization
-      (gethash dispatch cl--generic-dispatchers)
+      ;; We need `copy-sequence` here because this `dispatch' object might be
+      ;; modified by side-effect in `cl-generic-define-method' (bug#46722).
+      (gethash (copy-sequence dispatch) cl--generic-dispatchers)
     ;; (message "cl--generic-get-dispatcher (%S)" dispatch)
     (let* ((dispatch-arg (car dispatch))
            (generalizers (cdr dispatch))