Re: [PATCH v2 1/3] emacs: Add new option notmuch-search-hide-excluded

Subject: Re: [PATCH v2 1/3] emacs: Add new option notmuch-search-hide-excluded

Date: Sun, 07 May 2023 06:27:57 -0600

To: Mohsin Kaleem, notmuch@notmuchmail.org

Cc:

From: David Bremner


Mohsin Kaleem <mohkale@kisara.moe> writes:

> We could workaround this by changing the semantics of the variable. Make
> it notmuch-search-show-excluded and default to nil so nothing has to be
> changed in existing functions that call notmuch-search. Or we could
> switch to using cl-defun and have the default value substituted in the
> parameter list. I've never used cl-defun with interactive functions but
> long term I think that's the saner solution.

Thanks for the explanation. I think we can live with it as is for now,
and look into the cl-defun "upgrade" later.

I suspect it will be a bit confusing if we invert the sense of the
variable relative to the CLI argument.

d
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: