On Tue, 24 Jan 2012 10:46:57 -0800, Jameson Graef Rollins <jrollins at finestructure.net> wrote: > On Tue, 24 Jan 2012 12:53:38 +0000, David Edmondson <dme at dme.org> wrote: > > Instead, allow the caller to specify some parameters for the > > button. Rework `notmuch-show-insert-part-multipart/signed' and > > `notmuch-show-insert-part-multipart/encrypted' accordingly. > > Hi, David. I was thinking about this, and it seems to me that returning > the button itself is useful. I can imagine in the future that it might > be useful to be able to modify the button after you've created. Maybe > it's inconvenient to specify all button parameters at creation time. > > Is there a reason it's really necessary to make this change? Can't > callers just ignore the returned button if they don't care about it > further? I can see that maybe it's nice to be able to specify > parameters at creation time, but I'm not sure why that requires throwing > out the returned object as well.
Patches 2 and 3 in that series can result in the button not being inserted. -------------- 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/20120124/1d9e0cd5/attachment.pgp>