On Sunday 15 April 2012 16:22:06 H.J. Lu wrote: > On Sun, Apr 15, 2012 at 12:03 PM, Mike Frysinger <[email protected]> wrote: > > On Sunday 15 April 2012 14:17:13 H.J. Lu wrote: > >> Linux kernel v3.4 adds x32 support whose clock_t is long long. This > >> patch casts clock_t type to unsigned long for "%lu". > > > > shouldn't we cast it to long long then and use %llu ? > > It also works for x32. > > I don't think it makes a difference in practice. I doubt clock_t value > returns by those system calls can overflow 32bit integer.
famous last words! seriously though, if clock_t is known to be 64bits, then we should be decoding it accordingly. otherwise, i'd question why it was declared 64bits in the first place instead of keeping it as 32. -mike
signature.asc
Description: This is a digitally signed message part.
------------------------------------------------------------------------------ For Developers, A Lot Can Happen In A Second. Boundary is the first to Know...and Tell You. Monitor Your Applications in Ultra-Fine Resolution. Try it FREE! http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________ Strace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/strace-devel
