[Webkit-unassigned] [Bug 119785] Replace currentTime() with monotonicallyIncreasingTime() in WebCore

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Aug 15 10:34:21 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=119785





--- Comment #9 from Arunprasad Rajkumar <arurajku at cisco.com>  2013-08-15 10:33:54 PST ---
(From update of attachment 208798)
View in context: https://bugs.webkit.org/attachment.cgi?id=208798&action=review

Thanks for your comments, I will submit the new patch soon :)

>> Source/WTF/wtf/CurrentTime.h:55
>> +// measurement.
> 
> Typo: "currenTime".
> 
> It’s not enough to say "highly recommended", this doesn’t explain anything to the reader, and someone who uses the wrong function won’t see this comment anyway. Perhaps you could say something like "result of this function increases monotonically even when clock time goes back due to DST changes or NTP adjustments".

Thanks Alexey, I will correct this.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list