elharo opened a new issue, #65:
URL: https://github.com/apache/maven-scripting-plugin/issues/65
### 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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
