LGTM, thanks for cleaning this up.

(It's a pity that we need to pass around the language mode at all. Judging from the changes you had to do, the only reason really is our elimination of empty
scopes.)


https://codereview.chromium.org/1231343003/diff/20001/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/1231343003/diff/20001/src/parser.cc#newcode4043
src/parser.cc:4043: bool use_strict_const = is_strict(language_mode) ||
Does this perhaps already fix this other inconsistency you found as
well?

https://codereview.chromium.org/1231343003/

--
--
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