Re: [PATCH v2 4/5] cli: new crypto verify flag to handle verification

Subject: Re: [PATCH v2 4/5] cli: new crypto verify flag to handle verification

Date: Sat, 19 May 2012 09:28:57 -0700

To: Jani Nikula, Notmuch Mail

Cc:

From: Jameson Graef Rollins


On Sat, May 19 2012, Jani Nikula <jani@nikula.org> wrote:
>> +    /* decryption implies verification */
>> +    if (params.crypto.decrypt)
>> +	params.crypto.verify = TRUE;
>
> This does not change existing behaviour, only makes it more obvious
> (which is good), but this seems to be missing from the man page. I
> presume technically decryption doesn't have to imply verification, but
> it's probably a good thing. It should be documented, but does not have
> to be a part of this series.

Well that's very strange.  I was sure I mentioned that in the man page
back when I originally added decryption support.  But apparently I
didn't.  I'll submit a separate patch to fix the documentation.

jamie.
part-000.sig (application/pgp-signature)

Thread: