Revision: 2625 Author: [email protected] Date: Wed Aug 5 01:53:02 2009 Log: 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. Review URL: http://codereview.chromium.org/160636 http://code.google.com/p/v8/source/detail?r=2625 Modified: /branches/bleeding_edge/src/log.cc ======================================= --- /branches/bleeding_edge/src/log.cc Wed Aug 5 00:35:05 2009 +++ /branches/bleeding_edge/src/log.cc Wed Aug 5 01:53:02 2009 @@ -1151,8 +1151,8 @@ 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 -~----------~----~----~----~------~----~------~--~---
