diff options
| author | Glenn Morris | 2009-09-30 07:24:35 +0000 |
|---|---|---|
| committer | Glenn Morris | 2009-09-30 07:24:35 +0000 |
| commit | 494f4fc70c9dc567b3341d2cf04e0ceabaa69373 (patch) | |
| tree | 72f01d4272e8665f6f56cac5e24924cbcc58b91b | |
| parent | d275a5ed7b78c5d6e71a8708440062f3d8e8b9a6 (diff) | |
| download | emacs-494f4fc70c9dc567b3341d2cf04e0ceabaa69373.tar.gz emacs-494f4fc70c9dc567b3341d2cf04e0ceabaa69373.zip | |
(elint-init-form): Report declarations where the filename is not a string.
| -rw-r--r-- | lisp/ChangeLog | 5 | ||||
| -rw-r--r-- | lisp/emacs-lisp/elint.el | 15 |
2 files changed, 15 insertions, 5 deletions
diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 05837378097..a6c42130907 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog | |||
| @@ -1,10 +1,15 @@ | |||
| 1 | 2009-09-30 Glenn Morris <rgm@gnu.org> | 1 | 2009-09-30 Glenn Morris <rgm@gnu.org> |
| 2 | 2 | ||
| 3 | * cedet/semantic/analyze/fcn.el (semantic-scope-find): Fix declaration. | ||
| 4 | |||
| 3 | * cedet/semantic/bovine/gcc.el (semantic-gcc-setup): Replace runtime use | 5 | * cedet/semantic/bovine/gcc.el (semantic-gcc-setup): Replace runtime use |
| 4 | of CL function `remove-if-not'. | 6 | of CL function `remove-if-not'. |
| 5 | 7 | ||
| 6 | * emacs-lisp/authors.el (authors-ignored-files): Add "js2-mode.el". | 8 | * emacs-lisp/authors.el (authors-ignored-files): Add "js2-mode.el". |
| 7 | 9 | ||
| 10 | * emacs-lisp/elint.el (elint-init-form): Report declarations where the | ||
| 11 | filename is not a string. | ||
| 12 | |||
| 8 | 2009-09-29 Chong Yidong <cyd@stupidchicken.com> | 13 | 2009-09-29 Chong Yidong <cyd@stupidchicken.com> |
| 9 | 14 | ||
| 10 | * files.el (safe-local-eval-forms): Fix typo. | 15 | * files.el (safe-local-eval-forms): Fix typo. |
diff --git a/lisp/emacs-lisp/elint.el b/lisp/emacs-lisp/elint.el index 27e7a7f293a..59be145d7ec 100644 --- a/lisp/emacs-lisp/elint.el +++ b/lisp/emacs-lisp/elint.el | |||
| @@ -420,11 +420,16 @@ Return nil if there are no more forms, t otherwise." | |||
| 420 | ((eq (car form) 'autoload) | 420 | ((eq (car form) 'autoload) |
| 421 | (setq env (elint-env-add-func env (cadr (cadr form)) 'unknown))) | 421 | (setq env (elint-env-add-func env (cadr (cadr form)) 'unknown))) |
| 422 | ((eq (car form) 'declare-function) | 422 | ((eq (car form) 'declare-function) |
| 423 | (setq env (elint-env-add-func env (cadr form) | 423 | (setq env (elint-env-add-func |
| 424 | (if (or (< (length form) 4) | 424 | env (cadr form) |
| 425 | (eq (nth 3 form) t)) | 425 | (if (or (< (length form) 4) |
| 426 | 'unknown | 426 | (eq (nth 3 form) t) |
| 427 | (nth 3 form))))) | 427 | (unless (stringp (nth 2 form)) |
| 428 | (elint-error "Malformed declaration for `%s'" | ||
| 429 | (cadr form)) | ||
| 430 | t)) | ||
| 431 | 'unknown | ||
| 432 | (nth 3 form))))) | ||
| 428 | ((and (eq (car form) 'defalias) (listp (nth 2 form))) | 433 | ((and (eq (car form) 'defalias) (listp (nth 2 form))) |
| 429 | ;; If the alias points to something already in the environment, | 434 | ;; If the alias points to something already in the environment, |
| 430 | ;; add the alias to the environment with the same arguments. | 435 | ;; add the alias to the environment with the same arguments. |