From 34e13ce015232968731de2a9ec3440a08b0084b1 Mon Sep 17 00:00:00 2001 From: Lasse Collin Date: Thu, 12 Jan 2023 03:51:07 +0200 Subject: Tests: Silence warnings from -Wsign-conversion. Note that assigning an unsigned int to lzma_check doesn't warn on GNU/Linux x86-64 since the enum type is unsigned on that platform. The enum can be signed on some other platform though so it's best to use enumeration type lzma_check in these situations. --- tests/test_stream_flags.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'tests/test_stream_flags.c') diff --git a/tests/test_stream_flags.c b/tests/test_stream_flags.c index b8ec546d..26b4c613 100644 --- a/tests/test_stream_flags.c +++ b/tests/test_stream_flags.c @@ -70,7 +70,7 @@ test_lzma_stream_header_encode(void) #ifndef HAVE_ENCODERS assert_skip("Encoder support disabled"); #else - for (int i = 0; i < LZMA_CHECK_ID_MAX; i++) + for (lzma_check i = 0; i < LZMA_CHECK_ID_MAX; i++) stream_header_encode_helper(i); lzma_stream_flags flags = { @@ -152,7 +152,7 @@ test_lzma_stream_footer_encode(void) #ifndef HAVE_ENCODERS assert_skip("Encoder support disabled"); #else - for (int i = 0; i < LZMA_CHECK_ID_MAX; i++) + for (lzma_check i = 0; i < LZMA_CHECK_ID_MAX; i++) stream_footer_encode_helper(i); lzma_stream_flags flags = { @@ -223,7 +223,7 @@ test_lzma_stream_header_decode(void) #if !defined(HAVE_ENCODERS) || !defined(HAVE_DECODERS) assert_skip("Encoder or decoder support disabled"); #else - for (int i = 0; i < LZMA_CHECK_ID_MAX; i++) + for (lzma_check i = 0; i < LZMA_CHECK_ID_MAX; i++) stream_header_decode_helper(i); lzma_stream_flags flags = { @@ -319,7 +319,7 @@ test_lzma_stream_footer_decode(void) #if !defined(HAVE_ENCODERS) || !defined(HAVE_DECODERS) assert_skip("Encoder or decoder support disabled"); #else - for (int i = 0; i < LZMA_CHECK_ID_MAX; i++) + for (lzma_check i = 0; i < LZMA_CHECK_ID_MAX; i++) stream_footer_decode_helper(i); lzma_stream_flags flags = { @@ -422,7 +422,7 @@ test_lzma_stream_flags_compare(void) second.check = LZMA_CHECK_CRC32; // Check types must be equal - for (uint32_t i = 0; i < LZMA_CHECK_ID_MAX; i++) { + for (lzma_check i = 0; i < LZMA_CHECK_ID_MAX; i++) { first.check = i; if (i == second.check) assert_lzma_ret(lzma_stream_flags_compare(&first, -- cgit v1.2.3