[PATCH v2 02/14] cli/reply: push notmuch reply format abstraction lower in the stack
Jani Nikula
jani at nikula.org
Fri Sep 9 23:20:33 PDT 2016
On Sat, 10 Sep 2016, David Bremner <david at tethera.net> wrote:
> Jani Nikula <jani at nikula.org> writes:
>
>> - if (mime_node_open (ctx, message, &(params->crypto), &root) == NOTMUCH_STATUS_SUCCESS) {
>> - format_part_reply (root);
>> - talloc_free (root);
>> - }
>> + g_object_unref (G_OBJECT (reply));
>>
>> - notmuch_message_destroy (message);
>> + if (mime_node_open (ctx, message, ¶ms->crypto, &root) == NOTMUCH_STATUS_SUCCESS) {
>> + format_part_reply (root);
>> + talloc_free (root);
>
> I'm not sure I should admit this, but I'd have to check a book to make
> sure that ¶ms->crypto is the same as &(params->crypto)
Nah, you wouldn't have to check a book. If it weren't the same, it
wouldn't compile. :)
BR,
Jani.
More information about the notmuch
mailing list