RFR: 8151832: Improve exception messages in exceptions thrown by jigsaw code

Seán Coffey sean.coffey at oracle.com
Tue May 31 19:10:02 UTC 2016


Thanks Paul. Best to have consistent formatting. Webrev updated in place.

On the beginning quote observation, I wasn't too such myself. Presumably 
a typo. I've removed for now.

Regards,
Sean.

On 31/05/2016 19:48, Paul Benedict wrote:
> Hi Sean,
>
> I just have a few minor comments.
>
> Nearly all the new messages follow the message/colon/space/details 
> format. There are a few missing the space between the colon and details:
> *) ImageHeader:
> "jimage header not the correct size:"
>
> *) JrtPath
> throw new ProviderMismatchException("path class:"
>
> *) ConstructorFinder
> all new messages
>
> Some other observations:
> *) ImageLocation: why the beginning quote?
> throw new InternalError("\"Missing jimage attribute data");
>
> *) JrtFileSystem: extra space before colon
> "option class : " + option.getClass().getName());
>
> Cheers,
> Paul
>
> On Tue, May 31, 2016 at 12:57 PM, Seán Coffey <sean.coffey at oracle.com 
> <mailto:sean.coffey at oracle.com>> wrote:
>
>     I've gone ahead with a trimmed down webrev as per Alan's request.
>
>     new webrev :
>     http://cr.openjdk.java.net/~coffeys/webrev.8151832.v2/webrev/
>     <http://cr.openjdk.java.net/%7Ecoffeys/webrev.8151832.v2/webrev/>
>
>     Regards,
>     Sean.
>
>     On 16/05/2016 15:10, Alan Bateman wrote:
>
>
>
>         On 16/05/2016 14:45, Seán Coffey wrote:
>
>
>             On 16/05/16 13:51, Alan Bateman wrote:
>
>                 On 16/05/2016 13:44, Seán Coffey wrote:
>
>                     Some extra capturing of context in exception
>                     handling. I've re-based the original suggested
>                     patch and added some minor edits.
>
>                     https://bugs.openjdk.java.net/browse/JDK-8151832
>                     webrev :
>                     http://cr.openjdk.java.net/~coffeys/webrev.8151832/webrev/index.html
>                     <http://cr.openjdk.java.net/%7Ecoffeys/webrev.8151832/webrev/index.html>
>
>                 Would it be possible to leave out the changes to the
>                 source files in the module and loader directories for
>                 now? We have many changes to this code coming that
>                 replace parts of this code and I think would be better
>                 to do a pass over the exceptions in a few months once
>                 the code is more stable.
>
>             Yes, I can hold off. I figured such improvements might
>             help people while they adopt and set up modules but let's
>             cancel until the code stabilizes some more!
>
>         Thanks. The rest mostly look okay although I think several of
>         these exceptions in the jimage code need to re-examined - for
>         example IOOBE is thrown in several places where the root cause
>         must be a corrupt or truncated jimage file.
>
>         -Alan
>
>
>
>




More information about the core-libs-dev mailing list