On Fri 2017-12-29 10:30:00 -0400, David Bremner wrote: > Daniel Kahn Gillmor writes: > >> No, we should not support --decrypt=nostash for show or reply. The >> semantics of the display commands (show, reply) are such that they >> *never* modify the index or stash anything in there. The equivalent for >> the indexing (new, insert, reindex) commands' "--decrypt=nostash" in the >> display commands is simply "--decrypt=true". > > I'm not sure I completely agree, but its a trivial matter to add nostash > later if desired. And it's always easier to add API / command options > than to take them away. yes, true that we can always expand out, and it's more parsimonious to start small. :) It sounds like you were suggesting "--decrypt=nostash" as a synonym for "--decrypt=true" on show/reply, which i confess i didn't fully understand when i wrote my response. If it really makes you feel better to add the alias/synonym, i wouldn't block such a change. But I think the documentation is tricky to write (and trickier to read and trickier still to understand!) if "--decrypt=nostash" means the same thing as "--decrypt=true" in one context, but they mean different things in a different context. I'm still trying to aim for that sweet spot where the smallest possible API guides the user to sensible decisions while still understanding what's going on. :) --dkg