Re: [PATCH 4/5] commit: implement free_commit_graft
From: Stefan Beller <hidden>
Date: 2017-08-15 17:04:32
On Tue, Aug 15, 2017 at 4:49 AM, Patryk Obara [off-list ref] wrote: Here is a good place to explain why this is a good patch, (which is not immediately obvious to me at least).
quoted hunk ↗ jump to hunk
Signed-off-by: Patryk Obara <redacted> --- commit.c | 11 ++++++++--- commit.h | 1 + 2 files changed, 9 insertions(+), 3 deletions(-)diff --git a/commit.c b/commit.c index 499fb14..6a145f1 100644 --- a/commit.c +++ b/commit.c@@ -109,15 +109,20 @@ static int commit_graft_pos(const unsigned char *sha1) commit_graft_sha1_access); } +void free_commit_graft(struct commit_graft *graft) +{ + free(graft); +} + int register_commit_graft(struct commit_graft *graft, int ignore_dups) { int pos = commit_graft_pos(graft->oid.hash); if (0 <= pos) { if (ignore_dups) - free(graft); + free_commit_graft(graft); else { - free(commit_graft[pos]); + free_commit_graft(commit_graft[pos]); commit_graft[pos] = graft; } return 1;@@ -163,7 +168,7 @@ struct commit_graft *read_graft_line(struct strbuf *line) bad_graft_data: error("bad graft data: %s", buf); - free(graft); + free_commit_graft(graft); return NULL; }diff --git a/commit.h b/commit.h index baecc0a..c1b319f 100644 --- a/commit.h +++ b/commit.h@@ -247,6 +247,7 @@ struct commit_graft { }; typedef int (*each_commit_graft_fn)(const struct commit_graft *, void *); +void free_commit_graft(struct commit_graft *); struct commit_graft *read_graft_line(struct strbuf *line); int register_commit_graft(struct commit_graft *, int); struct commit_graft *lookup_commit_graft(const struct object_id *oid); --2.9.5