22 Mar
2016
22 Mar
'16
11:51 a.m.
On Tuesday 22 March 2016 02:55:22 Tina Ruchandani wrote:
Truly fixing this would require changing the debug print to print more than 8 digits and using a different specifier from %li.
Why not just change it to %lli, and a cast to s64? I don't think the format string or the number of digits is important here because it's just debug output that is normally disabled.
I also see an existing bug in the format string: "%08li.%08li" means we end up printing a six digit microsecond value with two leading zeros, which is rather confusing. I think using "%lld.%06ld" would be best here, or possibly "%lld.%09ld" to print the whole nanoseconds so we can skip the division.
Arnd