diff options
author | Carl Worth <cworth@cworth.org> | 2009-10-26 14:12:56 -0700 |
---|---|---|
committer | Carl Worth <cworth@cworth.org> | 2009-10-26 14:12:56 -0700 |
commit | ef3ab5781a3730e019f92517366029540a960cd0 (patch) | |
tree | bb40ef980076be11f72bc05562026abb5dee6562 /tags.c | |
parent | 1726c5c814aecd924849a0b91c82d420af945827 (diff) |
tags: Replace sort() and reset() with prepare_iterator().
The previous functions were always called together, so we might as
well just have one function for this. Also, the reset() name was
poor, and prepare_iterator() is much more descriptive.
Diffstat (limited to 'tags.c')
-rw-r--r-- | tags.c | 24 |
1 files changed, 18 insertions, 6 deletions
@@ -23,6 +23,7 @@ #include <glib.h> /* GList */ struct _notmuch_tags { + int sorted; GList *tags; GList *iterator; }; @@ -52,27 +53,38 @@ _notmuch_tags_create (void *ctx) talloc_set_destructor (tags, _notmuch_tags_destructor); + tags->sorted = 1; tags->tags = NULL; tags->iterator = NULL; return tags; } +/* Add a new tag to 'tags'. The tags object will create its own copy + * of the string. + * + * Note: The tags object will not do anything to prevent duplicate + * tags being stored, so the caller really shouldn't pass + * duplicates. */ void _notmuch_tags_add_tag (notmuch_tags_t *tags, const char *tag) { tags->tags = g_list_prepend (tags->tags, talloc_strdup (tags, tag)); + tags->sorted = 0; } +/* Prepare 'tag' for iteration. + * + * The internal creator of 'tags' should call this function before + * returning 'tags' to the user to call the public functions such as + * notmuch_tags_has_more, notmuch_tags_get, and notmuch_tags_advance. */ void -_notmuch_tags_sort (notmuch_tags_t *tags) +_notmuch_tags_prepare_iterator (notmuch_tags_t *tags) { - tags->tags = g_list_sort (tags->tags, (GCompareFunc) strcmp); -} + if (! tags->sorted) + tags->tags = g_list_sort (tags->tags, (GCompareFunc) strcmp); + tags->sorted = 1; -void -_notmuch_tags_reset (notmuch_tags_t *tags) -{ tags->iterator = tags->tags; } |