Core review request for 6378701 "(enum) Unclear purpose of EnumConstantNotPresentException "

Joseph D. Darcy Joe.Darcy at Sun.COM
Fri Aug 21 20:52:15 UTC 2009


Alan Bateman wrote:
> Joseph D. Darcy wrote:
>> Hello.
>>
>> Please review the patch below is clarify the use of certain exception 
>> types to address bug 6378701 "(enum) Unclear purpose of 
>> EnumConstantNotPresentException."  Five exceptions/errors can be 
>> thrown by the methods of the AnnotatedElement interface; those 
>> methods are used to retrieve annotations reflectively.  I've added 
>> cross references between the AnnotatedElement intefaces and the 
>> throwables it may throw.
>>
>> -Joe
> Looks okay to me (although maybe you don't need two links to 
> j.l.reflect.AnnotatedElement from each of the exceptions?).
>

I think calling out the connection in the "See also" area makes things 
clearer since the "See also" section stands apart from the text.

Thanks,

-Joe



More information about the core-libs-dev mailing list