Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(371)

Issue 12824: Change Windows daylight saving time calculations to... (Closed)

Created:
12 years ago by William Hesse
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Change Windows daylight saving time calculations to use C stdlib local time functions, not Win32 functions. Win32 time functions slowed down with Windows XP SP3. Change all platforms to use equivalent years between 2008 and 2035 for years outside the range 1970..2037. Committed: http://code.google.com/p/v8/source/detail?r=896

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -29 lines) Patch
M src/date-delay.js View 4 1 chunk +12 lines, -9 lines 0 comments Download
M src/platform-win32.cc View 1 2 3 4 4 chunks +30 lines, -20 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
William Hesse
Here is the change in the daylight saving time lookup code. I also have a ...
12 years ago (2008-12-01 13:50:02 UTC) #1
Mads Ager (chromium)
LGTM. Too bad that the more correct system call became that slow in SP3. :-( ...
12 years ago (2008-12-01 14:17:18 UTC) #2
Mike Belshe
lgtm I assume you ran this through the mozilla tests? I know we had trouble ...
12 years ago (2008-12-01 17:23:52 UTC) #3
William Hesse
Recommended changes made. Other changes also made for correctness. Submitting. Please review again.
12 years ago (2008-12-02 13:18:10 UTC) #4
Mads Ager (chromium)
12 years ago (2008-12-02 13:38:55 UTC) #5
LGTM.

Powered by Google App Engine
This is Rietveld 408576698