DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=36230>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36230

           Summary: [jci] Bug in JaninoJavaCompiler
           Product: Commons
           Version: unspecified
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Sandbox
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


When changing the implementation of JaninoJavaCompiler to Janino 2.3.8 I made a
small mistake (around line 90):

UnitCompiler uc = new UnitCompiler(unit, loader);

The second parameter is the class loader used for searching classes during
compiling the CompilationUnit. "loader" is the equivalent to the
ContextClassLoader. If the CompilationUnit is now dependent on another class
that should be loaded using the CompilingClassLoader it fails obviously. The
correct line must be:

UnitCompiler uc = new UnitCompiler(unit, this);

I think this issue does not need a patch, does it?

Joerg

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to