> private final ScriptEngine scriptEngine; > > /** > * Constructor > * @param engine > */ > - public ScriptEngineShellTable(String engine) { > - this.engine = engine; > - this.scriptEngine = scriptEngineFactory.getEngineByName(engine); > + public ScriptEngineShellTable(ScriptEngineManager scriptEngineManager, > String engine) { > + this.scriptEngine = scriptEngineManager.getEngineByName(engine);
> We'd better fail here instead of silently returning null? What would you suggestion? `IllegalStateException` or so? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-karaf/pull/79/files/6c68f0d234eae7732069b14ac47bad8c885eed27#r77950939