[webkit-reviews] review denied: [Bug 11184] fixing failed two failed mozilla Date reg tests on windows : [Attachment 10943] get correct GMT offset when DST is on

bugzilla-request-daemon at macosforge.org bugzilla-request-daemon at macosforge.org
Fri Oct 6 07:30:40 PDT 2006


Darin Adler <darin at apple.com> has denied Darin Adler <darin at apple.com>'s
request for review:
Bug 11184: fixing failed two failed mozilla Date reg tests on windows
http://bugs.webkit.org/show_bug.cgi?id=11184

Attachment 10943: get correct GMT offset when DST is on
http://bugs.webkit.org/attachment.cgi?id=10943&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
We should not be doing a const_cast like this. If we need to make a change like
this, then dateToMseconds should be changed to take a const tm*.

(And, in passing, "dateToMseconds" is a strangely-capitalized name; we should
fix that too. And it's very strange that isDST takes a const double& instead of
a double. But neither of those has anything to do with this patch.)



More information about the webkit-reviews mailing list