[PATCH 2/3] mailmap: use higher level string list functions
- From
Stefan Beller <sbeller@google.com>
- Date
- Nov 24, 2014, 21:22 UTC
- Message-ID
- <1416864124-15231-2-git-send-email-sbeller@google.com>
- In-Reply-To
- <1416864124-15231-1-git-send-email-sbeller@google.com>
No functional changes intended. This commit makes user of higher level and better documented functions of the string list API, so the code is more understandable.
Note that also the required computational amount should not change in principal as we need to look up the item no matter if it is already part of the list or not. Once looked up, insertion comes for free.
Signed-off-by: Stefan Beller <sbeller@google.com> --- mailmap.c | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/mailmap.c b/mailmap.c index 81890a6..f0df350 100644 --- a/mailmap.c +++ b/mailmap.c @@ -78,15 +78,10 @@ static void add_mapping(struct string_list *map, new_email = NULL; } - if ((index = string_list_find_insert_index(map, old_email, 1)) < 0) { - /* mailmap entry exists, invert index value */ - index = -1 - index; - me = (struct mailmap_entry *)map->items[index].util; + struct string_list_item *item = string_list_insert(map, old_email); + if (item->util) { + me = (struct mailmap_entry *)item->util; } else { - /* create mailmap entry */ - struct string_list_item *item; - - item = string_list_insert_at_index(map, index, old_email); me = xcalloc(1, sizeof(struct mailmap_entry)); me->namemap.strdup_strings = 1; me->namemap.cmp = namemap_cmp;
-- 2.2.0.rc3