UncheckedIOException

[elharo]

Affected version

HEAD

Bug description

There are several UncheckedIOExceptions for issues that can definitely happen, and that are not handled where they should be. E.g.

    protected Object eval(ScriptEngine engine, ScriptContext context) throws ScriptException {
        try (FileReader reader = new FileReader(scriptFile)) {
            return engine.eval(reader, context);
        } catch (IOException ex) {
            throw new UncheckedIOException(scriptFile + " caused:", ex);
        }
    }

This is risky and bug prone. Ideally these should be changed to IOException, but if API compatibility prevents that it looks like most could be replaced with a ScriptException instead.

[500ocean700-a11y]

hello is this a good first issue

[elharo]

no, it is not