On 16/02/2018 13:03, Michal Vala wrote:
Hi,
I'm working on JDK-8170120[1]. I have 2 working solutions, but I'm not
happy with neither one.
That IOException is thrown from jdk.internal.jimage.BasicImageReader,
which is in java.base module. Jimage is implemented in jdk.jlink
module (jdk.tools.jimage.JImageTask).
One solution is new exception NotValidJimageException which can be
thrown from BasicImageReader and catch and handled at JImageTask. Then
it's easy to return proper error message to the output.
Issue is that this new exception has to be public in java.base so
de-facto defining new jdk core api, which of course I don't want to.
Next option is leave there IOException, but give it some known message
and then handle this message in JImageTask.
This work also well, but "parsing" some message from exception is a
bit clumsy.
Why can't you just catch the IOException and adds the exception message
to the error message that jimage prints?
-Alan
