lgtm

https://codereview.chromium.org/1218473003/diff/1/src/flag-definitions.h
File src/flag-definitions.h (right):

https://codereview.chromium.org/1218473003/diff/1/src/flag-definitions.h#newcode207
src/flag-definitions.h:207: V(harmony_classes, "harmony classes (implies
object literal extension)") \
Maybe you already have a change in flight for this, but you might as
well remove this parenthetical comment while you're here.

https://codereview.chromium.org/1218473003/diff/1/test/mjsunit/regress/regress-parse-object-literal.js
File test/mjsunit/regress/regress-parse-object-literal.js (left):

https://codereview.chromium.org/1218473003/diff/1/test/mjsunit/regress/regress-parse-object-literal.js#oldcode28
test/mjsunit/regress/regress-parse-object-literal.js:28: // Flags:
--noharmony-classes --noharmony-object-literals
I take it this test is deleted because this configuration no longer
exists?

https://codereview.chromium.org/1218473003/

--
--
v8-dev mailing list
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to