author | Brandon Casey <drafnel@gmail.com> | |
Thu, 16 Jul 2009 21:25:19 +0000 (16:25 -0500) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Thu, 16 Jul 2009 22:09:21 +0000 (15:09 -0700) | ||
commit | b45a09c4b4a80ccc6eeb0f390ab8f4eb6f31c922 | |
tree | 68ddec4a335c467bc84624b018e71259d4e5ff24 | tree | snapshot |
parent | 9d33f7c22fda81f7be8a7aa1576e6d399d3b45ec | commit | diff |
sha1_name.c: avoid unnecessary strbuf_release
When we fall back to a standard for_each_reflog_ent() after failing to find
the nth branch switch (or if we had a short reflog) with the call to
for_each_recent_reflog_ent(), we do not need to free the memory allocated
for our strbuf's since a strbuf_reset() will be performed in
grab_nth_branch_switch() before assigning to the entry.
Plus, the strbuf_release() negates the non-zero hint we initially gave to
strbuf_init() just above these lines.
Signed-off-by: Brandon Casey <drafnel@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
When we fall back to a standard for_each_reflog_ent() after failing to find
the nth branch switch (or if we had a short reflog) with the call to
for_each_recent_reflog_ent(), we do not need to free the memory allocated
for our strbuf's since a strbuf_reset() will be performed in
grab_nth_branch_switch() before assigning to the entry.
Plus, the strbuf_release() negates the non-zero hint we initially gave to
strbuf_init() just above these lines.
Signed-off-by: Brandon Casey <drafnel@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
sha1_name.c | diff | blob | history |