diff options
| author | Mattias EngdegÄrd | 2019-12-04 11:37:26 +0100 |
|---|---|---|
| committer | Mattias EngdegÄrd | 2019-12-04 11:37:26 +0100 |
| commit | f5672b24a8510ee5c62b9e366e4ab6b1fde5ed77 (patch) | |
| tree | a8a057271b3fc2fe3db9b46da14ea6f950b8fa0c | |
| parent | 23053770449ba1af24961a11d803ae5e948130b6 (diff) | |
| download | emacs-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.el | 2 |
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, |