RFR(S+M) / 8022701 Accessibility checking: InvocationTargetException is thrown instead of IllegalAccessError
Christian Thalinger
christian.thalinger at oracle.com
Thu Sep 12 11:34:12 PDT 2013
On Sep 12, 2013, at 11:28 AM, David Chase <david.r.chase at oracle.com> wrote:
> New webrev, commented line removed:
> http://cr.openjdk.java.net/~drchase/8022701/webrev.03/
I think the change is good given that the tests work now. Is your test derived from the test in the bug report?
And it would be good if John could also have a quick look (just be to be sure).
-- Chris
>
> On 2013-09-12, at 1:53 PM, David Chase <david.r.chase at oracle.com> wrote:
>
>> I believe it produced extraneous output -- it was not clear to me that it was either necessary or useful to fully describe all the converted exceptions that lead to the defined exception being thrown. The commented line should have just been removed (I think).
>>
>> On 2013-09-12, at 1:24 PM, Christian Thalinger <christian.thalinger at oracle.com> wrote:
>>
>>> + // err.initCause(ex);
>>>
>>> Why is this commented?
>>>
>>> -- Chris
>
More information about the mlvm-dev
mailing list