From: Denys Vlasenko <vda.linux@googlemail.com>
Date: Mon, 16 Apr 2012 16:12:27 +0000 (+0200)
Subject: Stop using %h[h]u format specifiers
X-Git-Tag: v4.7~22
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=61d62cf9481f100f76f1e8a2dfe131f638566633;p=strace

Stop using %h[h]u format specifiers

This is needed for simplified printf, and reduces code size a bit.

* block.c (block_ioctl): Cast the value to unsinged and use %u
instead of using %hu.
* desc.c (sys_io_cancel): Likewise.
* resource.c (sys_sysinfo): Likewise.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
---

diff --git a/block.c b/block.c
index 1dee8bf3..ea571b6d 100644
--- a/block.c
+++ b/block.c
@@ -136,7 +136,7 @@ block_ioctl(struct tcb *tcp, long code, long arg)
 			if (syserror(tcp) || umove(tcp, arg, &val) < 0)
 				tprintf(", %#lx", arg);
 			else
-				tprintf(", %hu", val);
+				tprintf(", %u", (unsigned)val);
 		}
 		break;
 
@@ -223,10 +223,12 @@ block_ioctl(struct tcb *tcp, long code, long arg)
 			if (syserror(tcp) || umove(tcp, arg, &geo) < 0)
 				tprintf(", %#lx", arg);
 			else
-				tprintf(", {heads=%hhu, sectors=%hhu, "
-					"cylinders=%hu, start=%lu}",
-					geo.heads, geo.sectors,
-					geo.cylinders, geo.start);
+				tprintf(", {heads=%u, sectors=%u, "
+					"cylinders=%u, start=%lu}",
+					(unsigned)geo.heads,
+					(unsigned)geo.sectors,
+					(unsigned)geo.cylinders,
+					geo.start);
 		}
 		break;
 
@@ -248,10 +250,10 @@ block_ioctl(struct tcb *tcp, long code, long arg)
 			if (umove(tcp, arg, &buts) < 0)
 				tprintf(", %#lx", arg);
 			else
-				tprintf(", {act_mask=%hu, buf_size=%u, "
+				tprintf(", {act_mask=%u, buf_size=%u, "
 					"buf_nr=%u, start_lba=%" PRIu64 ", "
 					"end_lba=%" PRIu64 ", pid=%u}",
-					buts.act_mask, buts.buf_size,
+					(unsigned)buts.act_mask, buts.buf_size,
 					buts.buf_nr, buts.start_lba,
 					buts.end_lba, buts.pid);
 		}
diff --git a/desc.c b/desc.c
index 884f143c..f0c18903 100644
--- a/desc.c
+++ b/desc.c
@@ -950,10 +950,10 @@ sys_io_cancel(struct tcb *tcp)
 		tprintf("%lu, ", tcp->u_arg[0]);
 #ifdef HAVE_LIBAIO_H
 		if (umove(tcp, tcp->u_arg[1], &iocb) == 0) {
-			tprintf("{%p, %u, %hu, %hu, %d}, ",
+			tprintf("{%p, %u, %u, %u, %d}, ",
 				iocb.data, iocb.key,
-				iocb.aio_lio_opcode,
-				iocb.aio_reqprio, iocb.aio_fildes);
+				(unsigned)iocb.aio_lio_opcode,
+				(unsigned)iocb.aio_reqprio, iocb.aio_fildes);
 		} else
 #endif
 			tprints("{...}, ");
diff --git a/resource.c b/resource.c
index d7a34ef3..1b8f1a48 100644
--- a/resource.c
+++ b/resource.c
@@ -379,9 +379,9 @@ sys_sysinfo(struct tcb *tcp)
 				(long) si.totalram, (long) si.freeram);
 			tprintf("sharedram=%lu, bufferram=%lu} ",
 				(long) si.sharedram, (long) si.bufferram);
-			tprintf("totalswap=%lu, freeswap=%lu, procs=%hu}",
+			tprintf("totalswap=%lu, freeswap=%lu, procs=%u}",
 				(long) si.totalswap, (long) si.freeswap,
-				si.procs);
+				(unsigned)si.procs);
 		}
 	}
 	return 0;