[PATCH RFC] roll our own printf implementation

Denys Vlasenko dvlasenk at redhat.com
Fri Mar 30 20:08:36 UTC 2012


On 03/30/2012 08:24 PM, Mike Frysinger wrote:
> it's hard to say that having to add casts to arguments is an improvement.  are
> those really necessary?

Casts are necessary only when you print chars or shorts -
a rare operation. As you see, in the entire strace tree
only four tprintf calls needed tweaking.

Strictly speaking, casts may be not necessary - IIRC,
variadic function arguments are auto-promoted to int anyway.
But I prefer to not rely on my vague recollections
of C standards - I'd use casts instead.

It is possible to retain ability to understand %h[h]u,
but is it worth slowing down every tprintf for just four
cases where we use it?

-- 
vda




More information about the Strace-devel mailing list