diff options
| author | Jani Nikula <jani@nikula.org> | 2017-10-07 11:44:04 +0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2017-10-09 22:24:02 -0300 |
| commit | 0f314c0c99befea599a68bea51d759b4133efef6 (patch) | |
| tree | 6d7fa97122e87bf6dcdb221267ac052f48331f59 /tag-util.c | |
| parent | 54aef071590cb23f61da943daa29080cf7446696 (diff) | |
cli: convert notmuch_bool_t to stdbool
C99 stdbool turned 18 this year. There really is no reason to use our
own, except in the library interface for backward
compatibility. Convert the cli and test binaries to stdbool.
Diffstat (limited to 'tag-util.c')
| -rw-r--r-- | tag-util.c | 24 |
1 files changed, 12 insertions, 12 deletions
@@ -7,7 +7,7 @@ struct _tag_operation_t { const char *tag; - notmuch_bool_t remove; + bool remove; }; struct _tag_op_list_t { @@ -35,7 +35,7 @@ line_error (tag_parse_status_t status, } const char * -illegal_tag (const char *tag, notmuch_bool_t remove) +illegal_tag (const char *tag, bool remove) { if (*tag == '\0' && ! remove) return "empty tag forbidden"; @@ -84,7 +84,7 @@ parse_tag_line (void *ctx, char *line, /* Parse tags. */ while ((tok = strtok_len (tok + tok_len, " ", &tok_len)) != NULL) { - notmuch_bool_t remove; + bool remove; char *tag; /* Optional explicit end of tags marker. */ @@ -168,7 +168,7 @@ parse_tag_command_line (void *ctx, int argc, char **argv, if (argv[i][0] != '+' && argv[i][0] != '-') break; - notmuch_bool_t is_remove = argv[i][0] == '-'; + bool is_remove = argv[i][0] == '-'; const char *msg; msg = illegal_tag (argv[i] + 1, is_remove); @@ -215,7 +215,7 @@ makes_changes (notmuch_message_t *message, size_t i; notmuch_tags_t *tags; - notmuch_bool_t changes = FALSE; + bool changes = false; /* First, do we delete an existing tag? */ for (tags = notmuch_message_get_tags (message); @@ -239,11 +239,11 @@ makes_changes (notmuch_message_t *message, notmuch_tags_destroy (tags); if (changes) - return TRUE; + return true; /* Now check for adding new tags */ for (i = 0; i < list->count; i++) { - notmuch_bool_t exists = FALSE; + bool exists = false; if (list->ops[i].remove) continue; @@ -253,7 +253,7 @@ makes_changes (notmuch_message_t *message, notmuch_tags_move_to_next (tags)) { const char *cur_tag = notmuch_tags_get (tags); if (strcmp (cur_tag, list->ops[i].tag) == 0) { - exists = TRUE; + exists = true; break; } } @@ -264,9 +264,9 @@ makes_changes (notmuch_message_t *message, * but this is OK from a correctness point of view */ if (! exists) - return TRUE; + return true; } - return FALSE; + return false; } @@ -359,7 +359,7 @@ tag_op_list_create (void *ctx) int tag_op_list_append (tag_op_list_t *list, const char *tag, - notmuch_bool_t remove) + bool remove) { /* Make room if current array is full. This should be a fairly * rare case, considering the initial array size. @@ -387,7 +387,7 @@ tag_op_list_append (tag_op_list_t *list, * Is the i'th tag operation a remove? */ -notmuch_bool_t +bool tag_op_list_isremove (const tag_op_list_t *list, size_t i) { assert (i < list->count); |
