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