diff options
author | Don Gagne <don@thegagnes.com> | 2014-06-29 17:47:24 -0700 |
---|---|---|
committer | Don Gagne <don@thegagnes.com> | 2014-06-29 17:47:24 -0700 |
commit | 92adbe9216c96c53d1baa4eb1e14b4ede272c080 (patch) | |
tree | a5167e5c336b4f8fcb1550317c7e26da7bfedbfe /src/systemcmds/tests/test_file.c | |
parent | 28a31708f98eefa4ceb04617f2da3dd7892c99fa (diff) | |
download | px4-firmware-92adbe9216c96c53d1baa4eb1e14b4ede272c080.tar.gz px4-firmware-92adbe9216c96c53d1baa4eb1e14b4ede272c080.tar.bz2 px4-firmware-92adbe9216c96c53d1baa4eb1e14b4ede272c080.zip |
Fix compiler warnings
Diffstat (limited to 'src/systemcmds/tests/test_file.c')
-rw-r--r-- | src/systemcmds/tests/test_file.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/systemcmds/tests/test_file.c b/src/systemcmds/tests/test_file.c index 96be1e8df..86f23b485 100644 --- a/src/systemcmds/tests/test_file.c +++ b/src/systemcmds/tests/test_file.c @@ -102,7 +102,7 @@ test_file(int argc, char *argv[]) } /* perform tests for a range of chunk sizes */ - unsigned chunk_sizes[] = {1, 5, 8, 13, 16, 32, 33, 64, 70, 128, 133, 256, 300, 512, 555, 1024, 1500}; + int chunk_sizes[] = {1, 5, 8, 13, 16, 32, 33, 64, 70, 128, 133, 256, 300, 512, 555, 1024, 1500}; for (unsigned c = 0; c < (sizeof(chunk_sizes) / sizeof(chunk_sizes[0])); c++) { @@ -116,7 +116,7 @@ test_file(int argc, char *argv[]) uint8_t write_buf[chunk_sizes[c] + alignments] __attribute__((aligned(64))); /* fill write buffer with known values */ - for (int i = 0; i < sizeof(write_buf); i++) { + for (size_t i = 0; i < sizeof(write_buf); i++) { /* this will wrap, but we just need a known value with spacing */ write_buf[i] = i+11; } @@ -224,9 +224,6 @@ test_file(int argc, char *argv[]) return 1; } - /* compare value */ - bool compare_ok = true; - for (int j = 0; j < chunk_sizes[c]; j++) { if (read_buf[j] != write_buf[j]) { warnx("COMPARISON ERROR: byte %d: %u != %u", j, (unsigned int)read_buf[j], (unsigned int)write_buf[j]); |