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
