janitor: useless checks before free
Signed-off-by: Pierre Habouzit <madcoder@debian.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Pierre Habouzit committed
Jul 22, 2009 at 23:51 UTC
24deea5273709f3685f981c33468a0cead6cc147
2 files changed
+2
-3
pack-revindex.c
+1
-2
@@ -149,8 +149,7 @@ void discard_revindex(void)
149
if (pack_revindex_hashsz) {
150
int i;
151
for (i = 0; i < pack_revindex_hashsz; i++)
152
- if (pack_revindex[i].revindex)
153
- free(pack_revindex[i].revindex);
152
+ free(pack_revindex[i].revindex);
153
free(pack_revindex);
154
pack_revindex_hashsz = 0;
155
}
walker.c
+1
-1
@@ -245,7 +245,7 @@ void walker_targets_free(int targets, char **target, const char **write_ref)
245
{
246
while (targets--) {
247
free(target[targets]);
248
- if (write_ref && write_ref[targets])
248
+ if (write_ref)
249
free((char *) write_ref[targets]);
250
}
251
}