700375 - Context.compileFunction() allows code with multiple function definitions

Status: RESOLVED INACTIVE

(Rhino Graveyard :: Core, defect)

Description

[Marshall Scorcio]

Attachment: CompileFunctionTest.java

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_2) AppleWebKit/535.2 (KHTML, like Gecko) Chrome/15.0.874.106 Safari/535.2

Steps to reproduce:

Running Rhino 1.7R3 with Java 1.6.0_26 on OS X 10.7

Called Context.compileFunction() with code that contains two function definitions.
Something like: "function x() { return 'x'; } function y() { return 'y'; }"


Actual results:

No exception was thrown and the returned Function object was x(). y() seems to have been silently ignored.

I've attached some test Java code that demonstrates this happening.


Expected results:

In Rhino 1.6R7 an IllegalArgumentException was thrown. It seems like something similar should happen. 

A method with the name "compileFunction" should only allow the source code for a single function. Silently ignoring the rest of the source code is not very helpful.

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Closing. Bug management is now done here:
https://github.com/mozilla/rhino