[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, &params->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 &params->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