https://codereview.chromium.org/557023002/diff/1/src/v8natives.js
File src/v8natives.js (right):

https://codereview.chromium.org/557023002/diff/1/src/v8natives.js#newcode1092
src/v8natives.js:1092: if (IS_PRIVATE(name)) continue;
IS_PRIVATE is only true if IS_SYMBOL is also true so we can skip this
check for the common, string key case.

https://codereview.chromium.org/557023002/diff/1/src/v8natives.js#newcode1095
src/v8natives.js:1095: continue;
fits on previous line

V8 style is to always use {} if we can't fit on one line.

https://codereview.chromium.org/557023002/diff/1/src/v8natives.js#newcode1099
src/v8natives.js:1099: name = ToSTring(name);
typo

https://codereview.chromium.org/557023002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to