diff options
| author | Juanma Barranquero | 2006-10-05 15:53:30 +0000 |
|---|---|---|
| committer | Juanma Barranquero | 2006-10-05 15:53:30 +0000 |
| commit | 6b0a96341e8940314525e72ebe7f283ca3d90c82 (patch) | |
| tree | 2dbb783201b6b4768a0d41b0fa8c5d513441c7e2 | |
| parent | c2976c9c1973c7d0811ddb8c2bb427abfdfc03e3 (diff) | |
| download | emacs-6b0a96341e8940314525e72ebe7f283ca3d90c82.tar.gz emacs-6b0a96341e8940314525e72ebe7f283ca3d90c82.zip | |
(ad-remove-advice, ad-parse-arglist, ad-make-mapped-call):
Use `let', not `let*'.
| -rw-r--r-- | lisp/emacs-lisp/advice.el | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lisp/emacs-lisp/advice.el b/lisp/emacs-lisp/advice.el index d03245bf452..5aa8bbd14cc 100644 --- a/lisp/emacs-lisp/advice.el +++ b/lisp/emacs-lisp/advice.el | |||
| @@ -2409,7 +2409,7 @@ If such an advice was found it will be removed from the list of advices | |||
| 2409 | in that CLASS." | 2409 | in that CLASS." |
| 2410 | (interactive (ad-read-advice-specification "Remove advice of")) | 2410 | (interactive (ad-read-advice-specification "Remove advice of")) |
| 2411 | (if (ad-is-advised function) | 2411 | (if (ad-is-advised function) |
| 2412 | (let* ((advice-to-remove (ad-find-advice function class name))) | 2412 | (let ((advice-to-remove (ad-find-advice function class name))) |
| 2413 | (if advice-to-remove | 2413 | (if advice-to-remove |
| 2414 | (ad-set-advice-info-field | 2414 | (ad-set-advice-info-field |
| 2415 | function class | 2415 | function class |
| @@ -2747,7 +2747,7 @@ For that it has to be fbound with a non-autoload definition." | |||
| 2747 | A three-element list is returned, where the 1st element is the list of | 2747 | A three-element list is returned, where the 1st element is the list of |
| 2748 | required arguments, the 2nd is the list of optional arguments, and the 3rd | 2748 | required arguments, the 2nd is the list of optional arguments, and the 3rd |
| 2749 | is the name of an optional rest parameter (or nil)." | 2749 | is the name of an optional rest parameter (or nil)." |
| 2750 | (let* (required optional rest) | 2750 | (let (required optional rest) |
| 2751 | (setq rest (car (cdr (memq '&rest arglist)))) | 2751 | (setq rest (car (cdr (memq '&rest arglist)))) |
| 2752 | (if rest (setq arglist (reverse (cdr (memq '&rest (reverse arglist)))))) | 2752 | (if rest (setq arglist (reverse (cdr (memq '&rest (reverse arglist)))))) |
| 2753 | (setq optional (cdr (memq '&optional arglist))) | 2753 | (setq optional (cdr (memq '&optional arglist))) |
| @@ -2958,7 +2958,7 @@ Example: `(ad-map-arglists '(a &rest args) '(w x y z))' will return | |||
| 2958 | 2958 | ||
| 2959 | (defun ad-make-mapped-call (source-arglist target-arglist target-function) | 2959 | (defun ad-make-mapped-call (source-arglist target-arglist target-function) |
| 2960 | "Make form to call TARGET-FUNCTION with args from SOURCE-ARGLIST." | 2960 | "Make form to call TARGET-FUNCTION with args from SOURCE-ARGLIST." |
| 2961 | (let* ((mapped-form (ad-map-arglists source-arglist target-arglist))) | 2961 | (let ((mapped-form (ad-map-arglists source-arglist target-arglist))) |
| 2962 | (if (eq (car mapped-form) 'funcall) | 2962 | (if (eq (car mapped-form) 'funcall) |
| 2963 | (cons target-function (cdr (cdr mapped-form))) | 2963 | (cons target-function (cdr (cdr mapped-form))) |
| 2964 | (prog1 mapped-form | 2964 | (prog1 mapped-form |