From 7bc38acbf6d463a8d47c4b44e2dcb942f4df6049 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Thu, 15 May 2014 15:23:11 +0200 Subject: [PATCH] Polishing --- .../aop/interceptor/AsyncExecutionAspectSupport.java | 1 - .../aop/interceptor/AsyncExecutionInterceptor.java | 2 +- .../impl/MemorySafeUndeclaredThrowableStrategy.java | 8 ++++---- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionAspectSupport.java b/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionAspectSupport.java index 3d0840d0b77..3a1ed7959e3 100644 --- a/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionAspectSupport.java +++ b/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionAspectSupport.java @@ -21,7 +21,6 @@ import java.util.Map; import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.Executor; -import org.springframework.beans.BeansException; import org.springframework.beans.factory.BeanFactory; import org.springframework.beans.factory.BeanFactoryAware; import org.springframework.beans.factory.annotation.BeanFactoryAnnotationUtils; diff --git a/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionInterceptor.java b/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionInterceptor.java index bcd59da6f9d..569f8f8cfc9 100644 --- a/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionInterceptor.java +++ b/spring-aop/src/main/java/org/springframework/aop/interceptor/AsyncExecutionInterceptor.java @@ -63,7 +63,7 @@ public class AsyncExecutionInterceptor extends AsyncExecutionAspectSupport /** * Create a new {@code AsyncExecutionInterceptor}. * @param defaultExecutor the {@link Executor} (typically a Spring {@link AsyncTaskExecutor} - * or {@link java.util.concurrent.ExecutorService}) to delegate to. + * or {@link java.util.concurrent.ExecutorService}) to delegate to */ public AsyncExecutionInterceptor(Executor defaultExecutor) { super(defaultExecutor); diff --git a/spring-core/src/main/java/org/springframework/cglib/transform/impl/MemorySafeUndeclaredThrowableStrategy.java b/spring-core/src/main/java/org/springframework/cglib/transform/impl/MemorySafeUndeclaredThrowableStrategy.java index 6f20e3b9d26..decc7640017 100644 --- a/spring-core/src/main/java/org/springframework/cglib/transform/impl/MemorySafeUndeclaredThrowableStrategy.java +++ b/spring-core/src/main/java/org/springframework/cglib/transform/impl/MemorySafeUndeclaredThrowableStrategy.java @@ -43,7 +43,7 @@ public class MemorySafeUndeclaredThrowableStrategy extends DefaultGeneratorStrat }; - private Class wrapper; + private final Class wrapper; public MemorySafeUndeclaredThrowableStrategy(Class wrapper) { @@ -52,9 +52,9 @@ public class MemorySafeUndeclaredThrowableStrategy extends DefaultGeneratorStrat protected ClassGenerator transform(ClassGenerator cg) throws Exception { - ClassTransformer tr = new UndeclaredThrowableTransformer(wrapper); - tr = new MethodFilterTransformer(TRANSFORM_FILTER, tr); - return new TransformingClassGenerator(cg, tr); + ClassTransformer ct = new UndeclaredThrowableTransformer(this.wrapper); + ct = new MethodFilterTransformer(TRANSFORM_FILTER, ct); + return new TransformingClassGenerator(cg, ct); } }