[Webkit-unassigned] [Bug 9651] MSVCRT time functions are too strict

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=9651


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #9244|review?                     |review-
               Flag|                            |




------- Comment #4 from darin at apple.com  2006-07-10 01:00 PDT -------
(From update of attachment 9244)
Looks like a step in the right direction.

But why are all these functions entirely inlined? That doesn't seem right for
code as big as this. Lets put the implementation in the .cpp file.

In yearFromTime, it looks to me like the code would be better if instead of
calling floor in the year expression the code simply cast to (int). That does
the floor "for free" and we want year to be an int anyway. The only uses of it
are casted to int later and the only operation done subsequently is to subtract
1.

This looks great. Please fix DateExtras.h to not be all inline, and then lets
get that in!


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.