* emacs-lisp/checkdoc.el (checkdoc-proper-noun-regexp): Match noun

at end of sentence (my Bug#4818).

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index d353fdf7832598c9e3af9eccb7b38b28fa441af8..bb8c9a5325911a3f701ed7b380cef5d322441f7b 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2009-11-08  Kevin Ryde  <user42@zip.com.au>
+
+       * emacs-lisp/checkdoc.el (checkdoc-proper-noun-regexp): Match noun
+       at end of sentence (my Bug#4818).
+
 2009-11-08  Jared Finder  <jfinder@crypticstudios.com>
 
        * progmodes/compile.el (compilation-error-regexp-alist-alist):

diff --git a/lisp/emacs-lisp/checkdoc.el b/lisp/emacs-lisp/checkdoc.el
index 6b226be0b28ca01d4a787200431462c29e34acd7..7140c997283457892a6abc04865e8a39bf50f24d 100644 (file)
--- a/lisp/emacs-lisp/checkdoc.el
+++ b/lisp/emacs-lisp/checkdoc.el
@@ -331,12 +331,15 @@ This should be set in an Emacs Lisp file's local variables."
   "List of words (not capitalized) which should be capitalized.")
 
 (defvar checkdoc-proper-noun-regexp
+  ;; "[.!?]" is for noun at end of a sentence, since those chars
+  ;; are symbol syntax in emacs-lisp-mode and so don't match \\_>.
+  ;; The \" allows it to be the last sentence in a docstring too.
   (let ((expr "\\_<\\(")
        (l checkdoc-proper-noun-list))
     (while l
       (setq expr (concat expr (car l) (if (cdr l) "\\|" ""))
            l (cdr l)))
-    (concat expr "\\)\\_>"))
+    (concat expr "\\)\\(\\_>\\|[.!?][ \t\n\"]\\)"))
   "Regular expression derived from `checkdoc-proper-noun-regexp'.")
 
 (defvar checkdoc-common-verbs-regexp nil