https://codereview.chromium.org/366153002/diff/1180001/src/parser.h
File src/parser.h (right):
https://codereview.chromium.org/366153002/diff/1180001/src/parser.h#newcode672
src/parser.h:672: Scope** ad_hoc_eval_scope);
On 2014/09/10 13:39:28, rossberg wrote:
Why is this called ad_hoc_eval_scope everywhere, and not just
eval_scope?
Renamed to eval_scope.
It was called ad hoc because of this comment:
// TODO(mstarzinger): Global strict eval calls, need their own
scope
// as specified in ES5 10.4.2(3). The correct fix would be to
always
// add this scope in DoParseProgram(), but that requires
adaptations
// all over the code base, so we go with a quick-fix for now.
// In the same manner, we have to patch the parsing mode.
https://codereview.chromium.org/366153002/
--
--
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.
