Re: [PATCH v2 1/4] fast-export: trivial cleanup
From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:55:09
Felipe Contreras wrote:
Setting commit to commit is a no-op.
Wrong description. This should say: The code uses the idiom of assigning commit to itself to quench a "may be used uninitialized" warning. Luckily at least modern versions of gcc do not produce that warning here, so we can drop the self-assignment. This makes the code clearer to human beings, makes static analyzers that do not know that idiom happier, and means that if the code some day evolves to use this variable uninitialized then we will catch it.
Signed-off-by: Felipe Contreras <redacted>
With that change, for what it's worth, Reviewed-by: Jonathan Nieder <redacted> Patch left unsnipped since it doesn't seem to have hit the list.
quoted hunk
--- builtin/fast-export.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/builtin/fast-export.c b/builtin/fast-export.c index 12220ad..065f324 100644 --- a/builtin/fast-export.c +++ b/builtin/fast-export.c@@ -483,7 +483,7 @@ static void get_tags_and_duplicates(struct object_array *pending, for (i = 0; i < pending->nr; i++) { struct object_array_entry *e = pending->objects + i; unsigned char sha1[20]; - struct commit *commit = commit; + struct commit *commit; char *full_name; if (dwim_ref(e->name, strlen(e->name), sha1, &full_name) != 1)