[webkit-reviews] review denied: [Bug 5257] setYear() does not match FireFox/IE behavior : [Attachment 4193] Suggested patch

[bugzilla-request-daemon at opendarwin.org]

Geoffrey Garen <ggaren at apple.com> has denied Geoffrey Garen
<ggaren at apple.com>'s request for review:
Bug 5257: setYear() does not match FireFox/IE behavior
http://bugzilla.opendarwin.org/show_bug.cgi?id=5257

Attachment 4193: Suggested patch
http://bugzilla.opendarwin.org/attachment.cgi?id=4193&action=edit

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
I think it's a mistake to call fillStructures..., since that function parses
optional month and day arguments, which aren't valid in a setYear call. (That's
the cause for the r-.) 

I'm also not clear how this solves the 18XX case, since fillStructures... will
set tm_year to y - 1900 (a negative number).

Finally, it would be neat if you made this change in Date::construct() and
Date::callAsFunction, since they have the same problem.