Avoid unnecessary setAccessible call in AttributeMethods

Closes spring-projectsgh-23829

Author: jhoeller

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

@@ -25,6 +25,7 @@
 import org.springframework.lang.Nullable;
 import org.springframework.util.Assert;
 import org.springframework.util.ConcurrentReferenceHashMap;
+import org.springframework.util.ReflectionUtils;
 
 /**
  * Provides a quick way to access the attribute methods of an {@link Annotation}
@@ -73,15 +74,11 @@ private AttributeMethods(@Nullable Class<? extends Annotation> annotationType, M
 			if (method.getDefaultValue() != null) {
 				foundDefaultValueMethod = true;
 			}
-			if (type.isAnnotation() ||
-					(type.isArray() && type.getComponentType().isAnnotation())) {
+			if (type.isAnnotation() || (type.isArray() && type.getComponentType().isAnnotation())) {
 				foundNestedAnnotation = true;
 			}
-			method.setAccessible(true);
-			this.canThrowTypeNotPresentException[i] =
-					type == Class.class ||
-					type == Class[].class ||
-					type.isEnum();
+			ReflectionUtils.makeAccessible(method);
+			this.canThrowTypeNotPresentException[i] = (type == Class.class || type == Class[].class || type.isEnum());
 		}
 		this.hasDefaultValueMethod = foundDefaultValueMethod;
 		this.hasNestedAnnotation = foundNestedAnnotation;