<p><br>
The subject should be about what this patch does, like "lib: update notmuch_tags_get example to reflect api change"<br>
<br>
On Jan 29, 2012 12:46 AM, "Allan Wind" <<a href="mailto:allan_wind@lifeintegrity.com">allan_wind@lifeintegrity.com</a>> wrote:<br>
></p>
<p>And preferably you would describe here what the api change was.<br></p>
<p>> ---<br>
> lib/notmuch.h | 21 +++++++++++----------<br>
> 1 files changed, 11 insertions(+), 10 deletions(-)<br>
><br>
> diff --git a/lib/notmuch.h b/lib/notmuch.h<br>
> index 7929fe7..5e6e449 100644<br>
> --- a/lib/notmuch.h<br>
> +++ b/lib/notmuch.h<br>
> @@ -941,21 +941,22 @@ notmuch_message_get_header (notmuch_message_t *message, const char *header);<br>
> * Typical usage might be:<br>
> *<br>
> * notmuch_message_t *message;<br>
> + * notmuch_status_t status;<br>
> * notmuch_tags_t *tags;<br>
> * const char *tag;<br>
> *<br>
> - * message = notmuch_database_find_message (database, message_id);<br>
> - *<br>
> - * for (tags = notmuch_message_get_tags (message);<br>
> - * notmuch_tags_valid (tags);<br>
> - * notmuch_result_move_to_next (tags))<br>
> - * {<br>
> - * tag = notmuch_tags_get (tags);<br>
> - * ....<br>
> + * status = notmuch_database_find_message (database, message_id, &message);<br>
> + * if(!status && message) {<br>
> + * for (tags = notmuch_message_get_tags (message);<br>
> + * notmuch_tags_valid (tags);<br>
> + * notmuch_result_move_to_next (tags))<br>
> + * {<br>
> + * tag = notmuch_tags_get (tags);<br>
> + * ....<br>
> + * }<br>
> + * notmuch_message_destroy (message);<br>
> * }<br>
> *<br>
> - * notmuch_message_destroy (message);<br>
> - *<br>
> * Note that there's no explicit destructor needed for the<br>
> * notmuch_tags_t object. (For consistency, we do provide a<br>
> * notmuch_tags_destroy function, but there's no good reason to call<br>
> --<br>
> 1.7.2.5<br>
><br>
> _______________________________________________<br>
> notmuch mailing list<br>
> <a href="mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>
> <a href="http://notmuchmail.org/mailman/listinfo/notmuch">http://notmuchmail.org/mailman/listinfo/notmuch</a><br>
</p>