The msvc compiler thinks that several variables could be used
while uninitialised and issues 12 warnings like:
...\git\match-trees.c(75) : warning C4700: uninitialized local \
variable 'elem1' used
...\git\match-trees.c(76) : warning C4700: uninitialized local \
variable 'elem2' used
...\git\match-trees.c(77) : warning C4700: uninitialized local \
variable 'path1' used
...\git\match-trees.c(78) : warning C4700: uninitialized local \
variable 'path2' used
...\git\match-trees.c(79) : warning C4700: uninitialized local \
variable 'mode1' used
...\git\match-trees.c(80) : warning C4700: uninitialized local \
variable 'mode2' used
In order to suppress the warnings, we simply initialise the char
pointer variables to NULL and the int variables to zero.
Signed-off-by: Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxx>
---
match-trees.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/match-trees.c b/match-trees.c
index 26f7ed1..de050d0 100644
--- a/match-trees.c
+++ b/match-trees.c
@@ -72,12 +72,12 @@ static int score_trees(const unsigned char *hash1, const unsigned char *hash2)
die("%s is not a tree", sha1_to_hex(hash2));
init_tree_desc(&two, two_buf, size);
while (one.size | two.size) {
- const unsigned char *elem1 = elem1;
- const unsigned char *elem2 = elem2;
- const char *path1 = path1;
- const char *path2 = path2;
- unsigned mode1 = mode1;
- unsigned mode2 = mode2;
+ const unsigned char *elem1 = NULL;
+ const unsigned char *elem2 = NULL;
+ const char *path1 = NULL;
+ const char *path2 = NULL;
+ unsigned mode1 = 0;
+ unsigned mode2 = 0;
int cmp;
if (one.size)
--
1.7.3
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html