<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>