diff options
| author | Ramsay Jones <ramsay@ramsay1.demon.co.uk> | 2011-09-11 20:39:32 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-10-09 13:28:04 -0700 |
| commit | 27c0f768843b6d844674f1fb8dc2676d830b56e3 (patch) | |
| tree | 8f5e9fba34ffd5c042aee1b30118869feff13650 | |
| parent | 16f5bfcf65b46260f3a4fc66769e8b5cfd18a11a (diff) | |
| download | git-27c0f768843b6d844674f1fb8dc2676d830b56e3.tar.gz | |
Fix some "variable might be used uninitialized" warnings
In particular, gcc complains as follows:
CC tree-walk.o
tree-walk.c: In function `traverse_trees':
tree-walk.c:347: warning: 'e' might be used uninitialized in this \
function
CC builtin/revert.o
builtin/revert.c: In function `verify_opt_mutually_compatible':
builtin/revert.c:113: warning: 'opt2' might be used uninitialized in \
this function
Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | builtin/revert.c | 2 | ||||
| -rw-r--r-- | tree-walk.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/builtin/revert.c b/builtin/revert.c index ba27cf15ee..200149e2e7 100644 --- a/builtin/revert.c +++ b/builtin/revert.c @@ -110,7 +110,7 @@ static void verify_opt_compatible(const char *me, const char *base_opt, ...) static void verify_opt_mutually_compatible(const char *me, ...) { - const char *opt1, *opt2; + const char *opt1, *opt2 = NULL; va_list ap; va_start(ap, me); diff --git a/tree-walk.c b/tree-walk.c index 808bb55ba3..a8d8a66d59 100644 --- a/tree-walk.c +++ b/tree-walk.c @@ -344,7 +344,7 @@ int traverse_trees(int n, struct tree_desc *t, struct traverse_info *info) unsigned long mask, dirmask; const char *first = NULL; int first_len = 0; - struct name_entry *e; + struct name_entry *e = NULL; int len; for (i = 0; i < n; i++) { |