summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJim Meyering <meyering@redhat.com>2009-08-28 14:07:35 +0200
committerJim Meyering <meyering@redhat.com>2009-08-28 14:07:35 +0200
commitde81a7d930a6a2ad558eff9396da20237e06ccc1 (patch)
tree812f244e3de5aead34de1c2f92e4e028e1300fae
parentba5842c1147f4463fce7f613618938fd5b9a1c1a (diff)
downloadlibguestfs-de81a7d930a6a2ad558eff9396da20237e06ccc1.tar.gz
libguestfs-de81a7d930a6a2ad558eff9396da20237e06ccc1.tar.xz
libguestfs-de81a7d930a6a2ad558eff9396da20237e06ccc1.zip
generator.ml: avoid a warning about signed overflow in tests.c
* src/generator.ml: Emit "unsigned long int n_failed;" rather than "int failed;", to avoid warning from gcc about "assuming signed overflow does not occur when simplifying conditional to constant".
-rwxr-xr-xsrc/generator.ml8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/generator.ml b/src/generator.ml
index 569e1a13..2e2b70e5 100755
--- a/src/generator.ml
+++ b/src/generator.ml
@@ -5379,7 +5379,7 @@ static void print_table (char const *const *argv)
int main (int argc, char *argv[])
{
char c = 0;
- int failed = 0;
+ unsigned long int n_failed = 0;
const char *filename;
int fd;
int nr_tests, test_num = 0;
@@ -5513,7 +5513,7 @@ int main (int argc, char *argv[])
pr " printf (\"%%3d/%%3d %s\\n\", test_num, nr_tests);\n" test_name;
pr " if (%s () == -1) {\n" test_name;
pr " printf (\"%s FAILED\\n\");\n" test_name;
- pr " failed++;\n";
+ pr " n_failed++;\n";
pr " }\n";
) test_names;
pr "\n";
@@ -5524,8 +5524,8 @@ int main (int argc, char *argv[])
pr " unlink (\"test3.img\");\n";
pr "\n";
- pr " if (failed > 0) {\n";
- pr " printf (\"***** %%d / %%d tests FAILED *****\\n\", failed, nr_tests);\n";
+ pr " if (n_failed > 0) {\n";
+ pr " printf (\"***** %%lu / %%d tests FAILED *****\\n\", n_failed, nr_tests);\n";
pr " exit (1);\n";
pr " }\n";
pr "\n";