From 9ca1f945d9f5030600dc14ffff10d4dad14db4ca Mon Sep 17 00:00:00 2001 From: Jonas Bernoulli Date: Sun, 10 Jan 2021 15:01:07 +0100 Subject: [PATCH] emacs: improve how cl-lib and pcase are required We need to load `cl-lib' at run-time because we use more from it than just macros. Never-the-less many, but not all libraries required it only at compile-time, which we got away with because at least some libraries already required it at run-time as well. We use `cl-lib' and (currently to a lesser extend) `pcase' throughout the code-base, which means that we should require these features in most libraries. In the past we tried to only require these features in just the libraries that actually need them, without fully succeeding. We did not succeed in doing so because that means we would have to check every time that we use a function from these features whether they are already being required in the current library. An alternative would be to add the `require' forms at the top of every library but that is a bit annoying too. In order to make sure that these features are loaded when needed but also to keep the noise down we only require them in "notmuch-lib.el", which most other libraries require, and in most of the few libraries that do not do so, namely "notmuch-draft.el", "notmuch-message.el" and "notmuch-parser.el". ("coolj.el", "make-deps.el", various generated libraries, and "notmuch-compat.el" are left touched.) --- emacs/notmuch-address.el | 2 -- emacs/notmuch-company.el | 2 -- emacs/notmuch-draft.el | 3 +++ emacs/notmuch-hello.el | 1 - emacs/notmuch-jump.el | 4 ---- emacs/notmuch-lib.el | 1 + emacs/notmuch-maildir-fcc.el | 2 -- emacs/notmuch-message.el | 3 +++ emacs/notmuch-mua.el | 2 -- emacs/notmuch-parser.el | 3 ++- emacs/notmuch-show.el | 4 ---- emacs/notmuch-tag.el | 4 ---- emacs/notmuch-tree.el | 2 -- emacs/notmuch.el | 2 -- 14 files changed, 9 insertions(+), 26 deletions(-) diff --git a/emacs/notmuch-address.el b/emacs/notmuch-address.el index f313c415..ca24c744 100644 --- a/emacs/notmuch-address.el +++ b/emacs/notmuch-address.el @@ -21,8 +21,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'message) (require 'notmuch-parser) (require 'notmuch-lib) diff --git a/emacs/notmuch-company.el b/emacs/notmuch-company.el index 4439cc15..c6a004ae 100644 --- a/emacs/notmuch-company.el +++ b/emacs/notmuch-company.el @@ -32,8 +32,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'notmuch-lib) (defvar-local notmuch-company-last-prefix nil) diff --git a/emacs/notmuch-draft.el b/emacs/notmuch-draft.el index bc688434..2939da55 100644 --- a/emacs/notmuch-draft.el +++ b/emacs/notmuch-draft.el @@ -25,6 +25,9 @@ ;;; Code: +(require 'cl-lib) +(require 'pcase) + (require 'notmuch-maildir-fcc) (require 'notmuch-tag) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index 186ac172..24d2d19e 100644 --- a/emacs/notmuch-hello.el +++ b/emacs/notmuch-hello.el @@ -21,7 +21,6 @@ ;;; Code: -(require 'cl-lib) (require 'widget) (require 'wid-edit) ; For `widget-forward'. diff --git a/emacs/notmuch-jump.el b/emacs/notmuch-jump.el index 34d6c796..6fab5a79 100644 --- a/emacs/notmuch-jump.el +++ b/emacs/notmuch-jump.el @@ -22,10 +22,6 @@ ;;; Code: -(eval-when-compile - (require 'cl-lib) - (require 'pcase)) - (require 'notmuch-lib) (require 'notmuch-hello) diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el index cbac8859..6130309a 100644 --- a/emacs/notmuch-lib.el +++ b/emacs/notmuch-lib.el @@ -22,6 +22,7 @@ ;;; Code: (require 'cl-lib) +(require 'pcase) (require 'mm-util) (require 'mm-view) diff --git a/emacs/notmuch-maildir-fcc.el b/emacs/notmuch-maildir-fcc.el index 9a6365ba..ae8f5140 100644 --- a/emacs/notmuch-maildir-fcc.el +++ b/emacs/notmuch-maildir-fcc.el @@ -21,8 +21,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'message) (require 'notmuch-lib) diff --git a/emacs/notmuch-message.el b/emacs/notmuch-message.el index abeff53a..b90c934a 100644 --- a/emacs/notmuch-message.el +++ b/emacs/notmuch-message.el @@ -21,6 +21,9 @@ ;;; Code: +(require 'cl-lib) +(require 'pcase) + (require 'message) (require 'notmuch-tag) diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el index 2e4dc71a..c5b1b482 100644 --- a/emacs/notmuch-mua.el +++ b/emacs/notmuch-mua.el @@ -21,8 +21,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'message) (require 'mm-view) (require 'format-spec) diff --git a/emacs/notmuch-parser.el b/emacs/notmuch-parser.el index 294e0544..9749a6be 100644 --- a/emacs/notmuch-parser.el +++ b/emacs/notmuch-parser.el @@ -21,7 +21,8 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) +(require 'cl-lib) +(require 'pcase) (defun notmuch-sexp-create-parser () "Return a new streaming S-expression parser. diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el index 27925669..ea4444e5 100644 --- a/emacs/notmuch-show.el +++ b/emacs/notmuch-show.el @@ -23,10 +23,6 @@ ;;; Code: -(eval-when-compile - (require 'cl-lib) - (require 'pcase)) - (require 'mm-view) (require 'message) (require 'mm-decode) diff --git a/emacs/notmuch-tag.el b/emacs/notmuch-tag.el index 3c958dd4..982b372c 100644 --- a/emacs/notmuch-tag.el +++ b/emacs/notmuch-tag.el @@ -23,10 +23,6 @@ ;;; Code: -(require 'cl-lib) -(eval-when-compile - (require 'pcase)) - (require 'crm) (require 'notmuch-lib) diff --git a/emacs/notmuch-tree.el b/emacs/notmuch-tree.el index 51a43edd..95d5f642 100644 --- a/emacs/notmuch-tree.el +++ b/emacs/notmuch-tree.el @@ -24,8 +24,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'mail-parse) (require 'notmuch-lib) diff --git a/emacs/notmuch.el b/emacs/notmuch.el index 3436e1fc..d2e87b1b 100644 --- a/emacs/notmuch.el +++ b/emacs/notmuch.el @@ -65,8 +65,6 @@ ;;; Code: -(eval-when-compile (require 'cl-lib)) - (require 'mm-view) (require 'message)