On Thu 2019-05-23 22:13:59 -0300, David Bremner wrote: > Daniel Kahn Gillmor <[email protected]> writes: > >> diff --git a/emacs/notmuch-crypto.el b/emacs/notmuch-crypto.el >> index 353f721e..68171153 100644 >> --- a/emacs/notmuch-crypto.el >> +++ b/emacs/notmuch-crypto.el >> @@ -93,6 +93,7 @@ mode." >> (defun notmuch-crypto-insert-sigstatus-button (sigstatus from) >> (let* ((status (plist-get sigstatus :status)) >> (help-msg nil) >> + (show-button t) >> (label "Signature not processed") > > This should probably be nil, since that particular value is never used, > iiuc. I can amend it if you agree.
I don't think i understand your suggestion. that value *is* used, by the "(when show-button …)" construct later in the code. The idea was to make the button shown by default, unless it was disabled further down in the patch (in the "t" clause of the main "cond" construct in notmuch-crypto-insert-sigstatus-button, which only triggers if we legitimately don't have any status information (if "status" is nil)). So no, i think it's supposed to start off "t" and then get disabled in the specific condition this patch tests for. If i've misunderstood the e-lisp (entirely possible!) i would be happy to be corrected. --dkg
signature.asc
Description: PGP signature
_______________________________________________ notmuch mailing list [email protected] https://notmuchmail.org/mailman/listinfo/notmuch
