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

------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Strace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to