On 20/12/18 17:23 -0500, Nicholas Krause wrote:
This fixes the bug id, 71176 to use the proper known
code print formatter type, %lu for size_t rather than
%d which is considered best pratice for print statements.
Well the proper specifier for size_t is %zu, but since you cast to
unsigned long then %lu is right.
Signed-off-by: Nicholas Krause <xerofo...@gmail.com>
---
fixincludes/fixincl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fixincludes/fixincl.c b/fixincludes/fixincl.c
index 6dba2f6e830..5b8b77a77f0 100644
--- a/fixincludes/fixincl.c
+++ b/fixincludes/fixincl.c
@@ -158,11 +158,11 @@ main (int argc, char** argv)
if (VLEVEL( VERB_PROGRESS )) {
tSCC zFmt[] =
"\
-Processed %5d files containing %d bytes \n\
+Processed %5d files containing %lu bytes \n\
Applying %5d fixes to %d files\n\
Altering %5d of them\n";
- fprintf (stderr, zFmt, process_ct, ttl_data_size, apply_ct,
+ fprintf (stderr, zFmt, process_ct, (unsigned int long) ttl_data_size,
apply_ct,
I'd expect "unsigned long" not "unsigned int long".
