[PATCH 0/3] reworked crypto toggle, plus a couple of other toggles

[David Edmondson]

On Sun, 29 Jan 2012 12:06:57 -0800, Jameson Graef Rollins <jrollins at finestructure.net> wrote:
> > My inclination is to remove `notmuch-crypto-process-mime' altogether
> > (declared it an obsolete variable) and allow users to set a default
> > for `notmuch-show-process-crypto' directly, but that is not done in
> > this patchset while awaiting feedback.
> 
> I'm not sure I understand this.  `notmuch-show-process-crypto' *is* what
> sets the default for the `notmuch-show-process-crypto' buffer-local
> variable.  How would you change the current behavior or settings (beyond
> just a variable rename)?

The problem with the code as I posted can be seen if you imagine the
following ~/.emacs.el sequence:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/emacs-lisp
Size: 521 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120130/7898ab15/attachment.bin>
-------------- next part --------------

I'm sure that I can figure out either some `defcustom' goop or worse to
sort it out so that the initialisation happens correctly.

> > `notmuch-crypto-process-mime' is used only in notmuch-show.el, so the
> > setting really belongs there with an appropriate name.
> 
> I intentionally put this in a separate section in case there were
> eventually other needs for this setting beyond just show mode.  I
> personally think we should just leave it where it is, particularly
> since it's been there for a while.

Okay.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120130/7898ab15/attachment.pgp>