aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMattias EngdegÄrd2019-12-04 11:37:26 +0100
committerMattias EngdegÄrd2019-12-04 11:37:26 +0100
commitf5672b24a8510ee5c62b9e366e4ab6b1fde5ed77 (patch)
treea8a057271b3fc2fe3db9b46da14ea6f950b8fa0c
parent23053770449ba1af24961a11d803ae5e948130b6 (diff)
downloademacs-f5672b24a8510ee5c62b9e366e4ab6b1fde5ed77.tar.gz
emacs-f5672b24a8510ee5c62b9e366e4ab6b1fde5ed77.zip
Don't use the return value of 'push'
Although 'push' returns the modified list, it isn't actually documented to do so, so don't rely on it. * lisp/emacs-lisp/rx.el (rx--translate-any): Add progn.
-rw-r--r--lisp/emacs-lisp/rx.el2
1 files changed, 1 insertions, 1 deletions
diff --git a/lisp/emacs-lisp/rx.el b/lisp/emacs-lisp/rx.el
index 6fde27831a0..0dc6e198663 100644
--- a/lisp/emacs-lisp/rx.el
+++ b/lisp/emacs-lisp/rx.el
@@ -378,7 +378,7 @@ If NEGATED, negate the sense."
378 (let ((class (cdr (assq arg rx--char-classes)))) 378 (let ((class (cdr (assq arg rx--char-classes))))
379 (and class 379 (and class
380 (or (memq class classes) 380 (or (memq class classes)
381 (push class classes)))))) 381 (progn (push class classes) t))))))
382 (t (error "Invalid rx `any' argument: %s" arg)))) 382 (t (error "Invalid rx `any' argument: %s" arg))))
383 (let ((items 383 (let ((items
384 ;; Translate strings and conses into nonoverlapping intervals, 384 ;; Translate strings and conses into nonoverlapping intervals,