Polishing

spring-aop/src/main/java/org/springframework/aop/support/ControlFlowPointcut.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2019 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.
@@ -77,8 +77,7 @@ public class ControlFlowPointcut implements Pointcut, ClassFilter, MethodMatcher
 	}
 
 	/**
-	 * Subclasses can override this if it's possible to filter out
-	 * some candidate classes.
+	 * Subclasses can override this if it's possible to filter out some candidate classes.
 	 */
 	@Override
 	public boolean matches(Method method, Class<?> targetClass) {

spring-core/src/main/java/org/springframework/core/CollectionFactory.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2019 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.
@@ -145,7 +145,7 @@ public final class CollectionFactory {
 	 * Create the most appropriate collection for the given collection type.
 	 * <p>Delegates to {@link #createCollection(Class, Class, int)} with a
 	 * {@code null} element type.
-	 * @param collectionType the desired type of the target collection; never {@code null}
+	 * @param collectionType the desired type of the target collection (never {@code null})
 	 * @param capacity the initial capacity
 	 * @return a new collection instance
 	 * @throws IllegalArgumentException if the supplied {@code collectionType}
@@ -164,7 +164,7 @@ public final class CollectionFactory {
 	 * supplied {@code elementType} is an enum type matching type {@code E}.
 	 * As an alternative, the caller may wish to treat the return value as a
 	 * raw collection or collection of {@link Object}.
-	 * @param collectionType the desired type of the target collection; never {@code null}
+	 * @param collectionType the desired type of the target collection (never {@code null})
 	 * @param elementType the collection's element type, or {@code null} if unknown
 	 * (note: only relevant for {@link EnumSet} creation)
 	 * @param capacity the initial capacity
@@ -280,7 +280,7 @@ public final class CollectionFactory {
 	 * may wish to treat the return value as a raw map or map keyed by
 	 * {@link Object}. Similarly, type safety cannot be enforced if the
 	 * desired {@code mapType} is {@link MultiValueMap}.
-	 * @param mapType the desired type of the target map; never {@code null}
+	 * @param mapType the desired type of the target map (never {@code null})
 	 * @param keyType the map's key type, or {@code null} if unknown
 	 * (note: only relevant for {@link EnumMap} creation)
 	 * @param capacity the initial capacity

spring-core/src/main/java/org/springframework/util/ConcurrentReferenceHashMap.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2018 the original author or authors.
+ * Copyright 2002-2019 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.
@@ -574,7 +574,8 @@ public class ConcurrentReferenceHashMap<K, V> extends AbstractMap<K, V> implemen
 		 * @param allowResize if resizing is permitted
 		 */
 		protected final void restructureIfNecessary(boolean allowResize) {
-			boolean needsResize = (this.count > 0 && this.count >= this.resizeThreshold);
+			int currCount = this.count;
+			boolean needsResize = (currCount > 0 && currCount >= this.resizeThreshold);
 			Reference<K, V> ref = this.referenceManager.pollForPurge();
 			if (ref != null || (needsResize && allowResize)) {
 				lock();

spring-jdbc/src/main/java/org/springframework/jdbc/support/SQLStateSQLExceptionTranslator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2019 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.
@@ -58,32 +58,32 @@ public class SQLStateSQLExceptionTranslator extends AbstractFallbackSQLException
 
 
 	static {
-		BAD_SQL_GRAMMAR_CODES.add("07");	// Dynamic SQL error
-		BAD_SQL_GRAMMAR_CODES.add("21");	// Cardinality violation
-		BAD_SQL_GRAMMAR_CODES.add("2A");	// Syntax error direct SQL
-		BAD_SQL_GRAMMAR_CODES.add("37");	// Syntax error dynamic SQL
-		BAD_SQL_GRAMMAR_CODES.add("42");	// General SQL syntax error
-		BAD_SQL_GRAMMAR_CODES.add("65");	// Oracle: unknown identifier
-
-		DATA_INTEGRITY_VIOLATION_CODES.add("01");	// Data truncation
-		DATA_INTEGRITY_VIOLATION_CODES.add("02");	// No data found
-		DATA_INTEGRITY_VIOLATION_CODES.add("22");	// Value out of range
-		DATA_INTEGRITY_VIOLATION_CODES.add("23");	// Integrity constraint violation
-		DATA_INTEGRITY_VIOLATION_CODES.add("27");	// Triggered data change violation
-		DATA_INTEGRITY_VIOLATION_CODES.add("44");	// With check violation
-
-		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("08");	 // Connection exception
-		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("53");	 // PostgreSQL: insufficient resources (e.g. disk full)
-		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("54");	 // PostgreSQL: program limit exceeded (e.g. statement too complex)
-		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("57");	 // DB2: out-of-memory exception / database not started
-		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("58");	 // DB2: unexpected system error
-
-		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("JW");	 // Sybase: internal I/O error
-		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("JZ");	 // Sybase: unexpected I/O error
-		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("S1");	 // DB2: communication failure
-
-		CONCURRENCY_FAILURE_CODES.add("40");	// Transaction rollback
-		CONCURRENCY_FAILURE_CODES.add("61");	// Oracle: deadlock
+		BAD_SQL_GRAMMAR_CODES.add("07");  // Dynamic SQL error
+		BAD_SQL_GRAMMAR_CODES.add("21");  // Cardinality violation
+		BAD_SQL_GRAMMAR_CODES.add("2A");  // Syntax error direct SQL
+		BAD_SQL_GRAMMAR_CODES.add("37");  // Syntax error dynamic SQL
+		BAD_SQL_GRAMMAR_CODES.add("42");  // General SQL syntax error
+		BAD_SQL_GRAMMAR_CODES.add("65");  // Oracle: unknown identifier
+
+		DATA_INTEGRITY_VIOLATION_CODES.add("01");  // Data truncation
+		DATA_INTEGRITY_VIOLATION_CODES.add("02");  // No data found
+		DATA_INTEGRITY_VIOLATION_CODES.add("22");  // Value out of range
+		DATA_INTEGRITY_VIOLATION_CODES.add("23");  // Integrity constraint violation
+		DATA_INTEGRITY_VIOLATION_CODES.add("27");  // Triggered data change violation
+		DATA_INTEGRITY_VIOLATION_CODES.add("44");  // With check violation
+
+		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("08");  // Connection exception
+		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("53");  // PostgreSQL: insufficient resources (e.g. disk full)
+		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("54");  // PostgreSQL: program limit exceeded (e.g. statement too complex)
+		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("57");  // DB2: out-of-memory exception / database not started
+		DATA_ACCESS_RESOURCE_FAILURE_CODES.add("58");  // DB2: unexpected system error
+
+		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("JW");  // Sybase: internal I/O error
+		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("JZ");  // Sybase: unexpected I/O error
+		TRANSIENT_DATA_ACCESS_RESOURCE_CODES.add("S1");  // DB2: communication failure
+
+		CONCURRENCY_FAILURE_CODES.add("40");  // Transaction rollback
+		CONCURRENCY_FAILURE_CODES.add("61");  // Oracle: deadlock
 	}