aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Branham2019-02-02 09:45:11 -0600
committerAlex Branham2019-02-08 16:26:29 -0600
commit1e155dcc8dcbaed926a1574bc543d404d2859866 (patch)
tree9c96f8ffec384bc898065fda8f17e65001ed9f3d
parentb3092b2873829317de56043a8247ad2631d24d68 (diff)
downloademacs-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.el14
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.
2215This checks for style only, such as the first line, Commentary:, 2211This checks for style only, such as the first line, Commentary:,
2216Code:, and others referenced in the style guide." 2212Code:, 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)))