Looks good Andreas.

I can sponsor this.

cheers
/Joel

On 24 okt 2013, at 17:18, Andreas Lundblad <[email protected]> wrote:

> Hi,
> 
> Please review the fix for JDK-8027170 below.
> 
> Description:
> Class.getAnnotationsByType did not differentiate between annotations in 
> superclass and annotations in supersuperclass. This is however required by 
> the definition of associated annotations.
> 
> The patch addresses this issue by modifying AnnotationSupport so that it 
> traverses the class hierarchy when searching for annotations associated with 
> the given class.
> 
> The new test, InheritedAssociatedAnnotations.java, demonstrates the issue 
> (and passes after the patch has been applied).
> 
> Link to web review:
> http://cr.openjdk.java.net/~alundblad/8027170
> 
> Link to bug report:
> https://bugs.openjdk.java.net/browse/JDK-8027170
> 
> Original discussion:
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2013-October/022455.html
> 
> -- Andreas Lundblad
> 

Reply via email to