LGTM with a couple of nits in the ChangeLog.

http://codereview.chromium.org/5203002/diff/1/ChangeLog
File ChangeLog (right):

http://codereview.chromium.org/5203002/diff/1/ChangeLog#newcode4
ChangeLog:4: http://code.google.com/p/v8/issues/detail?id=931
We normally just write '(issue 931).'

http://codereview.chromium.org/5203002/diff/1/ChangeLog#newcode6
ChangeLog:6: Spilt the random number state between JavaScript and the
private API.
Spilt -> Split.

http://codereview.chromium.org/5203002/diff/1/ChangeLog#newcode14
ChangeLog:14: Provide more accurate results about used heap size via
Past tense in all items?

http://codereview.chromium.org/5203002/

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

Reply via email to