[COMMITTED] support: Fix printf format for TEST_COMPARE_STRING

Message ID 20181107131224.30856-1-adhemerval.zanella@linaro.org
State New
Headers show
Series
  • [COMMITTED] support: Fix printf format for TEST_COMPARE_STRING
Related show

Commit Message

Adhemerval Zanella Nov. 7, 2018, 1:12 p.m.
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(-)

-- 
2.17.1

Comments

Florian Weimer Nov. 7, 2018, 1:16 p.m. | #1
* Adhemerval Zanella:

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

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


Thanks; and sorry about that.

Florian

Patch

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);