Skip to content

Commit 3ae7f29

Browse files
committed
Polishing
1 parent 15624f3 commit 3ae7f29

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

spring-aspects/src/main/java/org/springframework/transaction/aspectj/AbstractTransactionAspect.aj

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2017 the original author or authors.
2+
* Copyright 2002-2020 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -59,7 +59,8 @@ public abstract aspect AbstractTransactionAspect extends TransactionAspectSuppor
5959

6060
@Override
6161
public void destroy() {
62-
clearTransactionManagerCache(); // An aspect is basically a singleton
62+
// An aspect is basically a singleton -> cleanup on destruction
63+
clearTransactionManagerCache();
6364
}
6465

6566
@SuppressAjWarnings("adviceDidNotMatch")

spring-expression/src/main/java/org/springframework/expression/spel/ast/AstUtils.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2018 the original author or authors.
2+
* Copyright 2002-2020 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -54,10 +54,9 @@ public static List<PropertyAccessor> getPropertyAccessorsToTry(
5454
}
5555
else {
5656
if (targetType != null) {
57-
int pos = 0;
5857
for (Class<?> clazz : targets) {
5958
if (clazz == targetType) { // put exact matches on the front to be tried first?
60-
specificAccessors.add(pos++, resolver);
59+
specificAccessors.add(resolver);
6160
}
6261
else if (clazz.isAssignableFrom(targetType)) { // put supertype matches at the end of the
6362
// specificAccessor list

0 commit comments

Comments
 (0)