diff options
| author | Alex Branham | 2019-02-02 09:45:11 -0600 |
|---|---|---|
| committer | Alex Branham | 2019-02-08 16:26:29 -0600 |
| commit | 1e155dcc8dcbaed926a1574bc543d404d2859866 (patch) | |
| tree | 9c96f8ffec384bc898065fda8f17e65001ed9f3d | |
| parent | b3092b2873829317de56043a8247ad2631d24d68 (diff) | |
| download | emacs-1e155dcc8dcbaed926a1574bc543d404d2859866.tar.gz emacs-1e155dcc8dcbaed926a1574bc543d404d2859866.zip | |
Fix byte compile warnings in checkdoc.el
* lisp/emacs-lisp/checkdoc.el (checkdoc-file-comments-engine): Fix
byte compile warnings by requiring lisp-mnt top-level, remove XEmacs
compatibility code.
Bug#34290
| -rw-r--r-- | lisp/emacs-lisp/checkdoc.el | 14 |
1 files changed, 2 insertions, 12 deletions
diff --git a/lisp/emacs-lisp/checkdoc.el b/lisp/emacs-lisp/checkdoc.el index c0da61a5892..dca2f169567 100644 --- a/lisp/emacs-lisp/checkdoc.el +++ b/lisp/emacs-lisp/checkdoc.el | |||
| @@ -174,6 +174,7 @@ | |||
| 174 | (require 'cl-lib) | 174 | (require 'cl-lib) |
| 175 | (require 'help-mode) ;; for help-xref-info-regexp | 175 | (require 'help-mode) ;; for help-xref-info-regexp |
| 176 | (require 'thingatpt) ;; for handy thing-at-point-looking-at | 176 | (require 'thingatpt) ;; for handy thing-at-point-looking-at |
| 177 | (require 'lisp-mnt) | ||
| 177 | 178 | ||
| 178 | (defvar compilation-error-regexp-alist) | 179 | (defvar compilation-error-regexp-alist) |
| 179 | (defvar compilation-mode-font-lock-keywords) | 180 | (defvar compilation-mode-font-lock-keywords) |
| @@ -2205,21 +2206,10 @@ News agents may remove it" | |||
| 2205 | ;; | 2206 | ;; |
| 2206 | (defvar generate-autoload-cookie) | 2207 | (defvar generate-autoload-cookie) |
| 2207 | 2208 | ||
| 2208 | (eval-when-compile (require 'lisp-mnt)) ; expand silly defsubsts | ||
| 2209 | (declare-function lm-summary "lisp-mnt" (&optional file)) | ||
| 2210 | (declare-function lm-section-start "lisp-mnt" (header &optional after)) | ||
| 2211 | (declare-function lm-section-end "lisp-mnt" (header)) | ||
| 2212 | |||
| 2213 | (defun checkdoc-file-comments-engine () | 2209 | (defun checkdoc-file-comments-engine () |
| 2214 | "Return a message list if this file does not match the Emacs standard. | 2210 | "Return a message list if this file does not match the Emacs standard. |
| 2215 | This checks for style only, such as the first line, Commentary:, | 2211 | This checks for style only, such as the first line, Commentary:, |
| 2216 | Code:, and others referenced in the style guide." | 2212 | Code:, and others referenced in the style guide." |
| 2217 | (if (featurep 'lisp-mnt) | ||
| 2218 | nil | ||
| 2219 | (require 'lisp-mnt) | ||
| 2220 | ;; Old XEmacs don't have `lm-commentary-mark' | ||
| 2221 | (if (and (not (fboundp 'lm-commentary-mark)) (fboundp 'lm-commentary)) | ||
| 2222 | (defalias 'lm-commentary-mark #'lm-commentary))) | ||
| 2223 | (save-excursion | 2213 | (save-excursion |
| 2224 | (let* ((f1 (file-name-nondirectory (buffer-file-name))) | 2214 | (let* ((f1 (file-name-nondirectory (buffer-file-name))) |
| 2225 | (fn (file-name-sans-extension f1)) | 2215 | (fn (file-name-sans-extension f1)) |
| @@ -2280,7 +2270,7 @@ Code:, and others referenced in the style guide." | |||
| 2280 | (if (or (not checkdoc-force-history-flag) | 2270 | (if (or (not checkdoc-force-history-flag) |
| 2281 | (file-exists-p "ChangeLog") | 2271 | (file-exists-p "ChangeLog") |
| 2282 | (file-exists-p "../ChangeLog") | 2272 | (file-exists-p "../ChangeLog") |
| 2283 | (and (fboundp 'lm-history-mark) (funcall #'lm-history-mark))) | 2273 | (lm-history-mark)) |
| 2284 | nil | 2274 | nil |
| 2285 | (progn | 2275 | (progn |
| 2286 | (goto-char (or (lm-commentary-mark) (point-min))) | 2276 | (goto-char (or (lm-commentary-mark) (point-min))) |