Polishing

Author: jhoeller

spring-context/src/main/java/org/springframework/context/support/GenericApplicationContext.java

@@ -111,12 +111,12 @@
  */
 public class GenericApplicationContext extends AbstractApplicationContext implements BeanDefinitionRegistry {
 
-	private static final Consumer<Builder> asCglibProxy = hint ->
+	private static final Consumer<Builder> asClassBasedProxy = hint ->
 			hint.withMembers(MemberCategory.INVOKE_DECLARED_CONSTRUCTORS,
 					MemberCategory.INVOKE_DECLARED_METHODS,
 					MemberCategory.DECLARED_FIELDS);
 
-	private static final Consumer<Builder> asCglibProxyTarget = hint ->
+	private static final Consumer<Builder> asProxiedUserClass = hint ->
 			hint.withMembers(MemberCategory.INTROSPECT_DECLARED_CONSTRUCTORS,
 					MemberCategory.INVOKE_DECLARED_METHODS);
 
@@ -445,14 +445,16 @@ private void preDetermineBeanTypes(RuntimeHints runtimeHints) {
 				for (SmartInstantiationAwareBeanPostProcessor bpp : bpps) {
 					beanType = bpp.determineBeanType(beanType, beanName);
 					if (Proxy.isProxyClass(beanType)) {
+						// A JDK proxy class needs an explicit hint
 						runtimeHints.proxies().registerJdkProxy(beanType.getInterfaces());
 					}
 					else {
+						// Potentially a CGLIB-generated subclass with reflection hints
 						Class<?> userClass = ClassUtils.getUserClass(beanType);
 						if (userClass != beanType) {
 							runtimeHints.reflection()
-									.registerType(beanType, asCglibProxy)
-									.registerType(userClass, asCglibProxyTarget);
+									.registerType(beanType, asClassBasedProxy)
+									.registerType(userClass, asProxiedUserClass);
 						}
 					}
 				}

spring-core/src/test/java/org/springframework/core/task/SimpleAsyncTaskExecutorTests.java

@@ -32,7 +32,7 @@
 class SimpleAsyncTaskExecutorTests {
 
 	@Test
-	void cannotExecuteWhenConcurrencyIsSwitchedOff() throws Exception {
+	void cannotExecuteWhenConcurrencyIsSwitchedOff() {
 		SimpleAsyncTaskExecutor executor = new SimpleAsyncTaskExecutor();
 		executor.setConcurrencyLimit(ConcurrencyThrottleSupport.NO_CONCURRENCY);
 		assertThat(executor.isThrottleActive()).isTrue();
@@ -41,13 +41,13 @@ void cannotExecuteWhenConcurrencyIsSwitchedOff() throws Exception {
 	}
 
 	@Test
-	void throttleIsNotActiveByDefault() throws Exception {
+	void throttleIsNotActiveByDefault() {
 		SimpleAsyncTaskExecutor executor = new SimpleAsyncTaskExecutor();
 		assertThat(executor.isThrottleActive()).as("Concurrency throttle must not default to being active (on)").isFalse();
 	}
 
 	@Test
-	void threadNameGetsSetCorrectly() throws Exception {
+	void threadNameGetsSetCorrectly() {
 		final String customPrefix = "chankPop#";
 		final Object monitor = new Object();
 		SimpleAsyncTaskExecutor executor = new SimpleAsyncTaskExecutor(customPrefix);
@@ -57,7 +57,7 @@ void threadNameGetsSetCorrectly() throws Exception {
 	}
 
 	@Test
-	void threadFactoryOverridesDefaults() throws Exception {
+	void threadFactoryOverridesDefaults() {
 		final Object monitor = new Object();
 		SimpleAsyncTaskExecutor executor = new SimpleAsyncTaskExecutor(runnable -> new Thread(runnable, "test"));
 		ThreadNameHarvester task = new ThreadNameHarvester(monitor);
@@ -66,7 +66,7 @@ void threadFactoryOverridesDefaults() throws Exception {
 	}
 
 	@Test
-	void throwsExceptionWhenSuppliedWithNullRunnable() throws Exception {
+	void throwsExceptionWhenSuppliedWithNullRunnable() {
 		assertThatIllegalArgumentException().isThrownBy(() ->
 				new SimpleAsyncTaskExecutor().execute(null));
 	}

spring-web/src/main/java/org/springframework/http/RequestEntity.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2021 the original author or authors.
+ * Copyright 2002-2022 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.
@@ -28,7 +28,6 @@
 import org.springframework.lang.Nullable;
 import org.springframework.util.MultiValueMap;
 import org.springframework.util.ObjectUtils;
-import org.springframework.web.util.UriTemplateHandler;
 
 /**
  * Extension of {@link HttpEntity} that also exposes the HTTP method and the
@@ -167,7 +166,7 @@ public HttpMethod getMethod() {
 	 * on how to expand template and encode the URI. In such cases, the
 	 * {@code URI} is prepared by the
 	 * {@link org.springframework.web.client.RestTemplate} with the help of the
-	 * {@link UriTemplateHandler} it is configured with.
+	 * {@link org.springframework.web.util.UriTemplateHandler} it is configured with.
 	 */
 	public URI getUrl() {
 		if (this.url == null) {