# HG changeset patch # User Stefan Monnier # Date 1003989099 0 # Node ID 7642548b1466de205b355664a6a6fa71ff1c2f98 # Parent 72bbea2ef3585f5c3bf7bdd16a26eb6bfa897060 (checkdoc-eval-defun): Call eval-defun interactively so that C-u M-C-x still does edebug. (checkdoc-sentencespace-region-engine): Don't force a double-space after `.' if it doesn't look like an end-of-sentence. (debug-ignored-errors): Add `disambiguate ...'. diff -r 72bbea2ef358 -r 7642548b1466 lisp/emacs-lisp/checkdoc.el --- a/lisp/emacs-lisp/checkdoc.el Thu Oct 25 05:22:30 2001 +0000 +++ b/lisp/emacs-lisp/checkdoc.el Thu Oct 25 05:51:39 2001 +0000 @@ -966,7 +966,7 @@ documentation is checked. If there is a documentation error, then the display of what was evaluated will be overwritten by the diagnostic message." (interactive) - (eval-defun nil) + (call-interactively 'eval-defun) (checkdoc-defun)) ;;;###autoload @@ -1654,7 +1654,7 @@ ;; Require whitespace OR ;; ITEMth OR ;; ITEMs - "\\(\\>\\|th\\>\\|s\\>\\)") + "\\(\\>\\|th\\>\\|s\\>\\|[.,;:]\\)") e t))) (if (not found) (let ((case-fold-search t)) @@ -2004,35 +2004,35 @@ (progn (set-syntax-table checkdoc-syntax-table) (goto-char begin) - (while (re-search-forward "[^.0-9]\\(\\. \\)[^ \n]" end t) + (while (re-search-forward "[^ .0-9]\\(\\. \\)[^ \n]" end t) (let ((b (match-beginning 1)) (e (match-end 1))) - (if (and (not (checkdoc-in-sample-code-p begin end)) - (not (checkdoc-in-example-string-p begin end)) - (not (save-excursion - (goto-char (match-beginning 1)) - (condition-case nil - (progn - (forward-sexp -1) - ;; piece of an abbreviation - (looking-at - "\\([a-z]\\|[iI]\\.?e\\|[eE]\\.?g\\)\\.")) - (error t))))) - (if (checkdoc-autofix-ask-replace - b e - "There should be two spaces after a period. Fix? " - ". ") - nil - (if errtxt - ;; If there is already an error, then generate - ;; the warning output if applicable - (if checkdoc-generate-compile-warnings-flag - (checkdoc-create-error - "There should be two spaces after a period" - b e)) - (setq errtxt - "There should be two spaces after a period" - bb b be e))))))) + (unless (or (checkdoc-in-sample-code-p begin end) + (checkdoc-in-example-string-p begin end) + (save-excursion + (goto-char b) + (condition-case nil + (progn + (forward-sexp -1) + ;; piece of an abbreviation + (looking-at + "\\([a-z]\\|[iI]\\.?e\\|[eE]\\.?g\\)\\.")) + (error t)))) + (if (checkdoc-autofix-ask-replace + b e + "There should be two spaces after a period. Fix? " + ". ") + nil + (if errtxt + ;; If there is already an error, then generate + ;; the warning output if applicable + (if checkdoc-generate-compile-warnings-flag + (checkdoc-create-error + "There should be two spaces after a period" + b e)) + (setq errtxt + "There should be two spaces after a period" + bb b be e))))))) (set-syntax-table old-syntax-table)) (if errtxt (checkdoc-create-error errtxt bb be)))))) @@ -2655,6 +2655,7 @@ (add-to-list 'debug-ignored-errors "Argument `.*' should appear (as .*) in the doc string") +(add-to-list 'debug-ignored-errors "Disambiguate .* by preceding .*") (provide 'checkdoc)