Add a test for |with (...) eval(...)| being direct

RESOLVED FIXED in mozilla2.0b7

Status

()

Core
JavaScript Engine
--
enhancement
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Waldo, Assigned: Waldo)

Tracking

Trunk
mozilla2.0b7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

I momentarily twisted my brain into knots thinking this case was an indirect eval, and I'd like to make sure we don't somehow, sometime, end up breaking this:

  var t = "global";
  function test()
  {
    var t = "local";
    with ({})
      return eval("t");
  }
  assertEq(test(), "local");

Updated

7 years ago
Attachment #480350 - Flags: review?(jimb) → review+
http://hg.mozilla.org/tracemonkey/rev/3a902a592341
Summary: Add a test for |with (global) eval(...)| being direct → Add a test for |with (...) eval(...)| being direct
Whiteboard: fixed-in-tracemonkey
Target Milestone: --- → mozilla2.0b8

Comment 3

7 years ago
http://hg.mozilla.org/mozilla-central/rev/3a902a592341
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in before you can comment on or make changes to this bug.