Polish annotation utils

spring-core/src/main/java/org/springframework/core/annotation/AbstractAliasAwareAnnotationAttributeExtractor.java

@@ -17,7 +17,6 @@
 package org.springframework.core.annotation;
 
 import java.lang.annotation.Annotation;
-import java.lang.reflect.AnnotatedElement;
 import java.lang.reflect.Method;
 import java.util.List;
 import java.util.Map;

spring-core/src/main/java/org/springframework/core/annotation/AnnotationAttributes.java

@@ -47,7 +47,7 @@ public class AnnotationAttributes extends LinkedHashMap<String, Object> {
 
 	private static final String UNKNOWN = "unknown";
 
-	private Class<? extends Annotation> annotationType;
+	private final Class<? extends Annotation> annotationType;
 
 	private final String displayName;
 
@@ -95,18 +95,23 @@ public class AnnotationAttributes extends LinkedHashMap<String, Object> {
 	 * or {@code null} to just store the annotation type name
 	 * @since 4.3.2
 	 */
-	@SuppressWarnings("unchecked")
 	public AnnotationAttributes(String annotationType, ClassLoader classLoader) {
 		Assert.notNull(annotationType, "'annotationType' must not be null");
+		this.annotationType = getAnnotationType(annotationType, classLoader);
+		this.displayName = annotationType;
+	}
+
+	@SuppressWarnings("unchecked")
+	private static Class<? extends Annotation> getAnnotationType(String annotationType, ClassLoader classLoader) {
 		if (classLoader != null) {
 			try {
-				this.annotationType = (Class<? extends Annotation>) classLoader.loadClass(annotationType);
+				return (Class<? extends Annotation>) classLoader.loadClass(annotationType);
 			}
 			catch (ClassNotFoundException ex) {
 				// Annotation Class not resolvable
 			}
 		}
-		this.displayName = annotationType;
+		return null;
 	}
 
 	/**

spring-core/src/main/java/org/springframework/core/annotation/AnnotationUtils.java

@@ -1129,11 +1129,10 @@ public abstract class AnnotationUtils {
 		// Only do defaults scanning for public annotations; we'd run into
 		// IllegalAccessExceptions otherwise, and we don't want to mess with
 		// accessibility in a SecurityManager environment.
-		Class<?> annotationType = attributes.annotationType();
+		Class<? extends Annotation> annotationType = attributes.annotationType();
 		if (annotationType != null && Modifier.isPublic(annotationType.getModifiers())) {
 			// Check declared default values of attributes in the annotation type.
-			Method[] annotationAttributes = annotationType.getMethods();
-			for (Method annotationAttribute : annotationAttributes) {
+			for (Method annotationAttribute : getAttributeMethods(annotationType)) {
 				String attributeName = annotationAttribute.getName();
 				Object defaultValue = annotationAttribute.getDefaultValue();
 				if (defaultValue != null && !attributes.containsKey(attributeName)) {

spring-core/src/main/java/org/springframework/core/annotation/DefaultAnnotationAttributeExtractor.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2015 the original author or authors.
+ * Copyright 2002-2016 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.