[COMMITTED] support: Fix printf format for TEST_COMPARE_STRING

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[COMMITTED] support: Fix printf format for TEST_COMPARE_STRING

Adhemerval Zanella-2
Fix the following on 32 bits targets:

support_test_compare_string.c: In function ‘support_test_compare_string’:
support_test_compare_string.c:80:37: error: format ‘%lu’ expects argument of
type ‘long unsigned int’, but argument 2 has type ‘size_t’ {aka ‘unsigned int’}
[-Werror=format=]
         printf ("  string length: %lu bytes\n", left_length);
                                   ~~^           ~~~~~~~~~~~
                                   %u
Checked on arm-linux-gnueabihf.

        * support/support_test_compare_string.c
        (support_test_compare_string): Fix printf format.
---
 ChangeLog                             | 5 +++++
 support/support_test_compare_string.c | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/support/support_test_compare_string.c b/support/support_test_compare_string.c
index 9958aaeec1..a76ba8eda7 100644
--- a/support/support_test_compare_string.c
+++ b/support/support_test_compare_string.c
@@ -77,7 +77,7 @@ support_test_compare_string (const char *left, const char *right,
       support_record_failure ();
       printf ("%s:%d: error: blob comparison failed\n", file, line);
       if (left_length == right_length && right != NULL && left != NULL)
-        printf ("  string length: %lu bytes\n", left_length);
+        printf ("  string length: %zu bytes\n", left_length);
       else
         {
           report_length ("left", left, left_length);
--
2.17.1

Reply | Threaded
Open this post in threaded view
|

Re: [COMMITTED] support: Fix printf format for TEST_COMPARE_STRING

Florian Weimer-5
* Adhemerval Zanella:

> -        printf ("  string length: %lu bytes\n", left_length);
> +        printf ("  string length: %zu bytes\n", left_length);

Thanks; and sorry about that.

Florian