LGTM with comment addressed.

http://codereview.chromium.org/2841041/diff/1/2
File src/date.js (right):

http://codereview.chromium.org/2841041/diff/1/2#newcode578
src/date.js:578: function DateValueGetDate(t) {
I don't like that (1) helper functions are now kinda mixed with
"interface" of Date specified by ECMA. (2) comments referencing ECMA
disappeared from functions like DateGetFullYear() etc.

http://codereview.chromium.org/2841041/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to