[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#30635: No compiler warning if code forgets to require cl-lib
From: |
Stefan Monnier |
Subject: |
bug#30635: No compiler warning if code forgets to require cl-lib |
Date: |
Mon, 19 Mar 2018 15:58:48 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
> There's no compiler warning if a library uses cl-lib without requiring it.
> I assume this is because bytecomp.el requires cl-lib.
Should we apply the patch below?
Stefan
diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index b3ea9300b0..e75403d80d 100644
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -124,17 +124,11 @@
(require 'backquote)
(require 'macroexp)
(require 'cconv)
-(require 'cl-lib)
-
-;; During bootstrap, cl-loaddefs.el is not created yet, so loading cl-lib
-;; doesn't setup autoloads for things like cl-every, which is why we have to
-;; require cl-extra as well (bug#18804).
-(or (fboundp 'cl-every)
- (require 'cl-extra))
-
-(or (fboundp 'defsubst)
- ;; This really ought to be loaded already!
- (load "byte-run"))
+(eval-when-compile
+ ;; We should refrain from loading cl-lib at run-time within the compiler
+ ;; code, otherwise we can't detect if a file forgets to (require 'cl-lib),
+ ;; as mentioned in bug#30635.
+ (require 'cl-lib))
;; The feature of compiling in a specific target Emacs version
;; has been turned off because compile time options are a bad idea.
@@ -3582,7 +3576,10 @@ byte-compile-and-folded
(cond
((< l 3) (byte-compile-form `(progn ,(nth 1 form) t)))
((= l 3) (byte-compile-two-args form))
- ((cl-every #'macroexp-copyable-p (nthcdr 2 form))
+ ;; This used to use `cl-every' but we need to avoid cl-lib at run-time as
+ ;; mentioned at the beginning of this file.
+ ((null (delq nil (mapcar (lambda (e) (not (macroexp-copyable-p e)))
+ (nthcdr 2 form))))
(byte-compile-form `(and (,(car form) ,(nth 1 form) ,(nth 2 form))
(,(car form) ,@(nthcdr 2 form)))))
(t (byte-compile-normal-call form)))))
- bug#30635: No compiler warning if code forgets to require cl-lib,
Stefan Monnier <=