http://codereview.chromium.org/266050/diff/8001/7002
File test/mjsunit/third_party/string-trim.js (right):

http://codereview.chromium.org/266050/diff/8001/7002#newcode81
Line 81: rightTrimString = wsString   + testString; // Trimmed from the
right.
You 'need' an extra space immediately preceding the double slash (//)
here.

http://codereview.chromium.org/266050/diff/8001/7002#newcode99
Line 99: var testValues = [0, Infinity, NaN, true, false, ({}),
['an','array']
Is there a missing comma at the end of this line?  I can't quite parse
this, but perhaps the machine can :-).

http://codereview.chromium.org/266050

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

Reply via email to