40485 - Patch to catch bytecode generated by javascript compiler

Status: RESOLVED FIXED

(Rhino Graveyard :: Compiler, enhancement, P3)

Description

[vajda]

- added ClassOutput interface
  - added setter/getter on Context and NameHelper (and its implementation).
  - instead of adding parameters in quite a few places, I followed the model
    set by Context.setTargetClassFileName() et al. by adding 
    Context.setClassOutput(). The classOutput gets used when a class is
    produced unless Context.setTargetClassFileName() was called.

Comment 1

[vajda]

Attachment: cvs diff -u

Comment 2

[vajda]

Attachment: Additional ClassOutput.java file (cvs add)

Comment 3

[vajda]

Attachment: An example ScriptCompiler.java file using the new ClassOutput feature

Comment 4

[Norris Boyd]

Reassign to my new account name.

Comment 5

[Norris Boyd]

Looks good. Regression tests pass with this change.

Since it introduces a new API class and we're trying to wrap up 1.5R1, I think 
I'll put this into 1.5R2.

Comment 6

[Norris Boyd]

I finally committed this. Thanks for your contribution!