js> function f(x) { s.exec(); } js> var s = new Script("print(x)"); js> f(3) js: "x" is not defined.

Roger, can you take this? I think we need to do for exec what we did for eval, although it's a little more complicated since exec is a method rather than a global function.

Fixed.

this is verified but i'll write a test for this.

the example above works okay. however, the following example does not: js> function f(x) { return s.exec(); } var s = new Script( "return x;"); var result = f(3); js> js: "<Script object>", line 1: invalid return js: return x; js: .....^ java.lang.NullPointerException: at org.mozilla.javascript.FunctionObject.construct (FunctionObject.java:445) at org.mozilla.javascript.ScriptRuntime.newObject (ScriptRuntime.java:1290) at c32.call(<stdin>:11) at c32.exec(<stdin>) at org.mozilla.javascript.Context.evaluateReader(Context.java:691) at org.mozilla.javascript.Context.evaluateString(Context.java:655) at org.mozilla.javascript.tools.shell.Main.processSource(Compiled Code) at org.mozilla.javascript.tools.shell.Main.main(Main.java:135)

Clearing Fixed resolution since bug has been set to Reopened. If this bug is actually fixed and verified, cbegle, I will leave it up to you to mark Verified/Fixed and perhaps open a new bug for the new behavior :-)

'var s = new Script( "return x;");' doesn't work because the script is not compiled within a function scope, even though it's going to be executed there, and so a return statement makes no sense. However, 'var s = new Script( "function y() { return x; } y(); ");' gives different answers on Rhino and Monkey. I'm going to have to ask a language lawyer to see which is right.

Script.exec() should *not* use the enclosing scope. Unfortunately, it currently does: js> s = new Script("print(a+1)"); function () { [native code] } js> a = 4 4 js> function f() { var a = 7 s.exec(); } js> f() 8 Calling f() should print out 5 rather than 8 on both engines.

Moving Rhino bugs to new Rhino product.

Fixed.

Trageting as resolved against 1.5R5

Untargeting wrong milestone