https://codereview.chromium.org/1226103002/diff/40001/test/mjsunit/harmony/block-conflicts-sloppy.js
File test/mjsunit/harmony/block-conflicts-sloppy.js (right):

https://codereview.chromium.org/1226103002/diff/40001/test/mjsunit/harmony/block-conflicts-sloppy.js#newcode62
test/mjsunit/harmony/block-conflicts-sloppy.js:62: //
TestAll('Conflict', 'eval("' + s + '");');
On 2015/07/08 23:42:13, Dan Ehrenberg wrote:
I don't see why these tests wouldn't make sense. Wouldn't there be a
conflict in
the new eval-driven block scope, even in sloppy mode?

See the bug.

  eval('let x; var x;')

is more similar to

  eval('{ let x; var x; }')

we are not creating the lexical block correctly for sloppy evals so
these tests are not passing or the tests are not correct for sloppy
mode.

https://codereview.chromium.org/1226103002/

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