Code review request for 8007113 Upgrade AnnotatedElement.isAnnotionPresent to be a default method
Joe Darcy
joe.darcy at oracle.com
Fri Feb 1 21:28:16 UTC 2013
On 02/01/2013 11:16 AM, Joe Darcy wrote:
> Hi Joel,
>
> On 2/1/2013 1:47 AM, Joel Borggrén-Franck wrote:
>> Hi Joe,
>>
>> On 02/01/2013 04:21 AM, Joe Darcy wrote:
>>> Hello,
>>>
>>> I'd like to upgrade the existing AnnotatedElement.isAnnotationPresent
>>> interface method to a default method:
>>>
>>> 8007113 Upgrade AnnotatedElement.isAnnotionPresent to be a
>>> default method
>>> http://cr.openjdk.java.net/~darcy/8007113.0/
>>>
>>
>> How about sun/reflect/generics/reflectiveObjects/TypeVariableImpl.java ?
>>
>> It looks like I forgot to update that one when I did the type
>> annotations changes :)
>
> I did notice TypeVariableImpl as one of the implementers of
> AnnotatedElement, but since it was defined to just return false, I
> skipped it. However, since getAnnotation now works for type
> variables, that behavior is incorrect.
>
> I'll update the webrev to cover TypeVariableImpl too.
>
> Thanks,
>
New webrev posted with a typo fix and TypeVariableImpl covered:
http://cr.openjdk.java.net/~darcy/8007113.1/
Thanks,
-Joe
More information about the core-libs-dev
mailing list