<p><br>
On Jan 7, 2012 5:52 AM, "David Bremner" <<a href="mailto:david@tethera.net">david@tethera.net</a>> wrote:<br>
><br>
> On Thu,  5 Jan 2012 22:25:12 +0200, Jani Nikula <<a href="mailto:jani@nikula.org">jani@nikula.org</a>> wrote:<br>
> > notmuch_show_params_t params is only initialized partially in<br>
> > notmuch_reply_command(). The only field that is used uninitialized is<br>
> > params.decrypt. It is usually non-zero, making "notmuch reply" on encrypted<br>
> > messages work by coincidence.<br>
><br>
> Hi Jani;<br>
><br>
> I get one test failure with this patch on current master:</p>
<p>Can't investigate right now, but did you try with just patch 1/4? (I really should have separated the bug fix from the rest.)</p>
<p>J.<br><br></p>
<p>><br>
>  FAIL   Reply within emacs<br>
>        --- emacs.24.expected   2012-01-07 03:47:50.000000000 +0000<br>
>        +++ emacs.24.output     2012-01-07 03:47:50.000000000 +0000<br>
>        @@ -1,5 +1,5 @@<br>
>         From: Notmuch Test Suite <<a href="mailto:test_suite@notmuchmail.org">test_suite@notmuchmail.org</a>><br>
>        -To: <a href="mailto:user@example.com">user@example.com</a><br>
>        +To:<br>
>         Subject: Re: Testing message sent via SMTP<br>
>         In-Reply-To: <XXX><br>
>         Fcc: /tmp/notmuch-dev-bremner/test/tmp.emacs/mail/sent<br>
><br>
> d<br>
</p>