jmx-dev Review Request: 7198070 Eliminate static dependency from JMX to java.beans.ConstructorProperties

[Alan Bateman]

On 18/09/2012 23:05, Mandy Chung wrote:
> Eamonn,
>
> I filed a RFE that the serviceability team can follow up your suggestion:
>    7199353: Allow ConstructorProperties annotation from any package
>
> Alan - I modified the fix to throw InternalError instead of 
> AssertionError.
It's great to have this static dependency removed. I should have been 
clearer in my comment about the AssertionError. What I meant is that I 
think we should looking at the InvocationTargetException's cause, and if 
an error or runtime exception then we should re-throw it rather than 
throwing AssertionError (or InternalError as it is now). That will 
making it easier to diagnose cases where the value method throws an 
error or a runtime exception.

-Alan