[PATCH] Use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
Re: [PATCH] Use ALLOC_GROW() instead of inline code · Michael Haggerty <hidden> · 2016-06-15
[PATCH v2 00/11] Use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 01/11] builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
Re: [PATCH v2 01/11] builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW() · Duy Nguyen <hidden> · 2016-06-15
Re: [PATCH v2 01/11] builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW() · Duy Nguyen <hidden> · 2016-06-15
Re: [PATCH v2 01/11] builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW() · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v2 01/11] builtin/pack-objects.c: change check_pbase_path() to use ALLOC_GROW() · Duy Nguyen <hidden> · 2016-06-15
[PATCH v2 02/11] bundle.c: change add_to_ref_list() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 03/11] cache-tree.c: change find_subtree() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 04/11] commit.c: change register_commit_graft() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 05/11] diff.c: use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 06/11] diffcore-rename.c: use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 07/11] patch-ids.c: change add_commit() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 08/11] replace_object.c: change register_replace_object() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 09/11] reflog-walk.c: use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
Re: [PATCH v2 09/11] reflog-walk.c: use ALLOC_GROW() instead of inline code · Duy Nguyen <hidden> · 2016-06-15
[PATCH v2 10/11] dir.c: change create_simplify() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v2 11/11] attr.c: change handle_attr_line() to use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
Re: [PATCH v2 00/11] Use ALLOC_GROW() instead of inline code · Michael Haggerty <hidden> · 2016-06-15
Re: [PATCH v2 00/11] Use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 00/11] Use ALLOC_GROW() instead of inline code · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 01/11] builtin/pack-objects.c: use ALLOC_GROW() in check_pbase_path() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 02/11] bundle.c: use ALLOC_GROW() in add_to_ref_list() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 03/11] cache-tree.c: use ALLOC_GROW() in find_subtree() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 04/11] commit.c: use ALLOC_GROW() in register_commit_graft() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 05/11] diff.c: use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 06/11] diffcore-rename.c: use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 07/11] patch-ids.c: use ALLOC_GROW() in add_commit() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 08/11] replace_object.c: use ALLOC_GROW() in register_replace_object() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 09/11] reflog-walk.c: use ALLOC_GROW() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 10/11] dir.c: use ALLOC_GROW() in create_simplify() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
[PATCH v3 11/11] attr.c: use ALLOC_GROW() in handle_attr_line() · Dmitry S. Dolzhenko <hidden> · 2016-06-15
Re: [PATCH v3 00/11] Use ALLOC_GROW() instead of inline code · Eric Sunshine <hidden> · 2016-06-15
Re: [PATCH] Use ALLOC_GROW() instead of inline code · He Sun <hidden> · 2016-06-15

DORMANTno replies

Revision v3 of 3 in this series.

Revisions (3)

2016-06-15 v2 [diff vs current]
2016-06-15 v3 current
2016-06-15 v4 [diff vs current]

[PATCH v3 07/11] patch-ids.c: use ALLOC_GROW() in add_commit()

From: Dmitry S. Dolzhenko <hidden>
Date: 2016-06-15 23:00:09
Subsystem: the rest · Maintainer: Linus Torvalds

Signed-off-by: Dmitry S. Dolzhenko <redacted>
---
 patch-ids.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/patch-ids.c b/patch-ids.c
index bc8a28f..bf81b92 100644
--- a/patch-ids.c
+++ b/patch-ids.c

@@ -83,10 +83,7 @@ static struct patch_id *add_commit(struct commit *commit,
 	ent = &bucket->bucket[bucket->nr++];
 	hashcpy(ent->patch_id, sha1);
 
-	if (ids->alloc <= ids->nr) {
-		ids->alloc = alloc_nr(ids->nr);
-		ids->table = xrealloc(ids->table, sizeof(ent) * ids->alloc);
-	}
+	ALLOC_GROW(ids->table, ids->nr + 1, ids->alloc);
 	if (pos < ids->nr)
 		memmove(ids->table + pos + 1, ids->table + pos,
 			sizeof(ent) * (ids->nr - pos));

-- 
1.8.5.3