Re: [PATCH v2 00/36] [emacs] Add outline headings and switch to lexical scope

Subject: Re: [PATCH v2 00/36] [emacs] Add outline headings and switch to lexical scope

Date: Wed, 13 Jan 2021 11:13:23 +0200

To: Jonas Bernoulli, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Sun, Jan 10 2021, Jonas Bernoulli wrote:

> This fixes a minor whitespace bug that Tomi notices in (1).
>
> This also adds for commits, two of them in response to a concern
> raised by Tomi, who didn't "see enough point for requiring subr-x
> just [for string-empty-p]", which I agree with:
>
> 2) Changes how `cl-lib' and `pcase' are required.  I did that first
>    because I want to do it the same way as for `subr-x'.
>
> 3) Require `subr-x', so that we can use it without having to worry
>    whether we have now reached the threshold where it becomes
>    justified to require an additional library.
>
>    Personally I consider this library to be part of core elisp
>    libraries, and wish it were autoloaded like, say `subr'.  I am
>    not the only one with that opinion and some other package is
>    bound to load this very small library anyways, so we might as
>    well benefit from the goodies that it provides too.
>
> The remaining two new commits (4,5) I have already submitted earlier
> in a separate thread.  Unfortunately they did not get merged or even
> just discussed so far, so I am including them here again.

Series looks good (also patch 35/36 (*)). It was easy to review, although
took time to browse through all small changes. Marked ready using 
*-notmuch::needs-review in notmuch show buffer. 

I have been using the series for couple of days now (and sending this
email having the changes in use) -- it feels like this starts faster
than previously but that must be just a feeling.

Tomi

(*) I would even suggest "the best thing", but then it would need more
discussion...

>
>      Cheers,
>      Jonas
>
> Jonas Bernoulli (36):
>   emacs: use setq instead of set
>   emacs: sanitize dedicated widget action/notify functions
>   emacs: define new notmuch-search-item widget type
>   emacs: notmuch-start-notmuch: remove backward compatibility code
>   emacs: notmuch-start-notmuch-error-sentinel: assert buffer is alive
>   emacs: notmuch-start-notmuch-sentinel: assert buffer is alive
>   emacs: notmuch-start-notmuch: avoid storing process buffer twice
>   emacs: avoid passing around some redundant information
>   emacs: avoid killing process buffer when process is still alive
>   emacs: make headings outline-minor-mode compatible
>   emacs: use lexical-bindings in all libraries
>   emacs: deal with unused lexical arguments and variables
>   emacs: notmuch-tag--get-formats: silence byte-compiler
>   emacs: inline notmuch-sexp-eof into only caller
>   emacs: notmuch-wash-region-to-button: remove unused MSG argument
>   emacs: silence compiler wrt notmuch-show-insert-part-text/plain
>   emacs: define notmuch-message-queued-tag-changes as buffer-local
>   emacs: notmuch-message-apply-queued-tag-changes: cosmetics
>   emacs: notmuch-wash.el: require diff-mode at beginning of code
>   emacs: notmuch-mua-prompt-for-sender: don't force Ido on users
>   emacs: notmuch-mua.el: move all options into "Options" section
>   emacs: notmuch-crypto-status-button-type: fix potential bug
> 1 emacs: various cosmetic improvements
>   emacs: various comment improvements
>   emacs: various doc-string improvements
>   emacs: remove variable notmuch-search-disjunctive-regexp
>   emacs: define a few variables as automatically buffer-local
>   emacs: notmuch-search-stash-thread-id: use notmuch-search-query-string
>   emacs: reorder notmuch.el a bit
>   emacs: avoid unnecessary let-bindings
> 2 emacs: improve how cl-lib and pcase are required
> 3 emacs: make subr-x available in all libraries
>   emacs: use string-empty-p
>   emacs: notmuch-tree-get-match: No longer define as command
> 4 emacs: allow opting out of notmuch's address completion
> 5 emacs: notmuch-address-expand-name: use the actual initial-input
>
>  emacs/coolj.el               |  14 +-
>  emacs/make-deps.el           |   2 +-
>  emacs/notmuch-address.el     | 103 +++++++-------
>  emacs/notmuch-company.el     |   3 -
>  emacs/notmuch-compat.el      |   4 +-
>  emacs/notmuch-crypto.el      |  14 +-
>  emacs/notmuch-draft.el       |  27 +++-
>  emacs/notmuch-hello.el       | 229 +++++++++++++++---------------
>  emacs/notmuch-jump.el        |  18 +--
>  emacs/notmuch-lib.el         | 199 +++++++++++++-------------
>  emacs/notmuch-maildir-fcc.el | 114 +++++++--------
>  emacs/notmuch-message.el     |  25 ++--
>  emacs/notmuch-mua.el         | 124 +++++++++--------
>  emacs/notmuch-parser.el      |  22 ++-
>  emacs/notmuch-print.el       |  16 ++-
>  emacs/notmuch-query.el       |  21 ++-
>  emacs/notmuch-show.el        | 130 ++++++++++-------
>  emacs/notmuch-tag.el         | 103 ++++++++------
>  emacs/notmuch-tree.el        |  61 ++++----
>  emacs/notmuch-wash.el        |  54 ++++----
>  emacs/notmuch.el             | 261 ++++++++++++++++++-----------------
>  emacs/rstdoc.el              |   2 +-
>  test/test-lib.el             |   4 +-
>  23 files changed, 813 insertions(+), 737 deletions(-)
>
> -- 
> 2.29.1
> _______________________________________________
> notmuch mailing list -- notmuch@notmuchmail.org
> To unsubscribe send an email to notmuch-leave@notmuchmail.org
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: