diff options
author | René Scharfe <rene.scharfe@lsrfire.ath.cx> | 2007-01-04 19:33:36 +0100 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2007-01-04 22:28:46 -0800 |
commit | 4fe2cc0c8906ba6d39c4fab89fe88dcf2226b184 (patch) | |
tree | ae247b354d25e7226ac065244edf0f65f310d536 /tree-walk.c | |
parent | 7c4c9f4cd9c36694ce0de5674f263585064cceec (diff) | |
download | git-4fe2cc0c8906ba6d39c4fab89fe88dcf2226b184.tar.gz git-4fe2cc0c8906ba6d39c4fab89fe88dcf2226b184.tar.xz |
Remove shadowing variable from traverse_trees()
The variable named entry is allocated using malloc() and then
forgotten, it being shadowed by an automatic variable of the
same name. Fixing the array size at 3 worked so far because
the only caller of traverse_trees() needed only as much
entries. Simply remove the shadowing varaible and we're able
to traverse more than three trees and save stack space at the
same time!
Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'tree-walk.c')
-rw-r--r-- | tree-walk.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tree-walk.c b/tree-walk.c index 14cc5aea6..22f4550b6 100644 --- a/tree-walk.c +++ b/tree-walk.c @@ -113,7 +113,6 @@ void traverse_trees(int n, struct tree_desc *t, const char *base, traverse_callb struct name_entry *entry = xmalloc(n*sizeof(*entry)); for (;;) { - struct name_entry entry[3]; unsigned long mask = 0; int i, last; |