Threading issues w/ the script bundle

[Mike Thomsen <mi...@gmail.com>]

I know there were some tricky threading issues with the scripting bundle,
so before I even go too far into trying another attempt at Kotlin support I
wanted to see if this looks like a sane approach based on those issues:

public class KotlinScriptEngineConfigurator extends
AbstractModuleClassloaderConfigurator {
    @Override
    public String getScriptEngineName() {
        return "kotlin";
    }

    private ScriptEngine scriptEngine;

    @Override
    public Object init(ScriptEngine engine, String[] modulePaths) throws
ScriptException {
        scriptEngine = engine;
        compiledForm = null;
        return scriptEngine;
    }

    private String currentScript;
    private CompiledScript compiledForm;

    private synchronized void compile(String scriptBody) {
        try {
            compiledForm = ((Compilable)scriptEngine).compile(scriptBody);
            currentScript = scriptBody;
        } catch (ScriptException e) {
            throw new RuntimeException(e);
        }
    }

    @Override
    public Object eval(ScriptEngine engine, String scriptBody, String[]
modulePaths) throws ScriptException {
        if (currentScript == null || compiledForm == null ||
!currentScript.equals(scriptBody)) {
            compile(scriptBody);
        }
        return compiledForm.eval();
    }
}