JCI-53: Stack overflow on cross imports in commons-jci-janino

Mark Proctor added a comment - 21/Sep/07 9:17 PM the current JCI implementation is not correctly handling all problems, for instance ParseExceptions are being thrown up and not placed into the problems array. I've since added this to the compiler.compile method, but I can't help feeling this should be handled elsewhere. Am I correct in assuming that IOException and CompileException are correctly handled, what about ScanException, is that thrown too without being added to the problems array. This is what I've added to JaninoJavaCompiler for now. try { compiler.compile(resources); } catch ( ScanException e ) { problems.add(new JaninoCompilationProblem(e)); // is this already handled? } catch ( ParseException e ) { problems.add(new JaninoCompilationProblem(e)); // we know this isn't handled } catch ( IOException e ) { // I'm hoping the existing compiler problems handler catches these } catch ( CompileException e ) { // I'm hoping the existing compiler problems handler catches these }

Mark Proctor added a comment - 21/Sep/07 9:20 PM sorry with better formatting this time: This is what I've added to JaninoJavaCompiler for now. try { compiler.compile(resources); } catch ( ScanException e ) { problems.add( new JaninoCompilationProblem(e)); // is this already handled? } catch ( ParseException e ) { problems.add( new JaninoCompilationProblem(e)); // we know this isn't handled } catch ( IOException e ) { // I'm hoping the existing compiler problems handler catches these } catch ( CompileException e ) { // I'm hoping the existing compiler problems handler catches these }

Arno Unkrig added a comment - 25/Dec/07 3:35 PM Ping JCI-53, waiting for reaction.

Matt Fowles added a comment - 23/Dec/09 9:38 AM This is a very old bug... I poked the people at https://issues.apache.org/jira/browse/JCI-53 to see if anything still needs to be done.

Matt Fowles added a comment - 13/Apr/10 9:49 AM I am closing this ticket as "overcome by events", also known as long dead