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

[Mandy Chung]

That’d be fine.

Mandy

> On Sep 22, 2016, at 11:43 AM, Sean Coffey <sean.coffey at oracle.com> wrote:
> 
> Thanks Mandy. I pushed this change earlier today. If BasicImageReader.java is being edited again in the near future, we might be able to make your suggested edits then.
> 
> regards,
> Sean.
> 
> On 22/09/2016 19:23, Mandy Chung wrote:
>>> On Sep 21, 2016, at 8:56 AM, Seán Coffey <sean.coffey at oracle.com> wrote:
>>> 
>>> Resurrecting this old review thread. After some internal discussion, I've dropped the minor edit that was made in StackTraceElementCompositeData. It could be noisy data for exception purposes. I've corrected the other issues raised by Alan and Jim has long pushed the changes mentioned below.
>>> 
>>> webrev : http://cr.openjdk.java.net/~coffeys/webrev.8151832.v3/webrev/
>> The change looks okay.
>> 
>> src/java.base/share/classes/jdk/internal/jimage/BasicImageReader.java
>>  189             throw new IOException("The image file \"" + name + "\" is not " +
>>  190                 "the correct version. Major: " + result.getMajorVersion() +
>>  191                 ". Minor: " + result.getMinorVersion());
>> 
>> One suggestion for this exception message to:
>> 
>>   throw new IOException("The image file \"" + name + "\” version “ +
>>      result.getMajorVersion() + "." + result.getMinorVersion() +
>>      “ not supported”);
>> 
>> No need for a new webrev and you can update this before you push.
>> 
>> Mandy
>