Reviewers: Søren Gjesse, Description: Fix issue 417: incorrect %t placeholder expansion.
BUG=http://code.google.com/p/v8/issues/detail?id=417 TEST=none Verified on Linux, Mac, and Windows. Please review this at http://codereview.chromium.org/160636 Affected files: M src/log.cc Index: src/log.cc diff --git a/src/log.cc b/src/log.cc index 44e5c32f13594a169a2571b36c4975154ce9643d..56808202add7186a395403310fb00df07a72567e 100644 --- a/src/log.cc +++ b/src/log.cc @@ -1151,8 +1151,8 @@ bool Logger::Setup() { break; case 't': { // %t expands to the current time in milliseconds. - uint32_t time = static_cast<uint32_t>(OS::TimeCurrentMillis()); - stream.Add("%u", time); + double time = OS::TimeCurrentMillis(); + stream.Add("%.0f", FmtElm(time)); break; } case '%': --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
