Deprecate outdated abstractions/delegates in core/util

Issue: SPR-15159

Author: jhoeller

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2015 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -22,8 +22,6 @@
 import org.springframework.aop.ClassFilter;
 import org.springframework.aop.MethodMatcher;
 import org.springframework.aop.Pointcut;
-import org.springframework.core.ControlFlow;
-import org.springframework.core.ControlFlowFactory;
 import org.springframework.util.Assert;
 import org.springframework.util.ObjectUtils;
 
@@ -34,7 +32,7 @@
  *
  * @author Rod Johnson
  * @author Rob Harrop
- * @see org.springframework.core.ControlFlow
+ * @author Juergen Hoeller
  */
 @SuppressWarnings("serial")
 public class ControlFlowPointcut implements Pointcut, ClassFilter, MethodMatcher, Serializable {
@@ -43,7 +41,7 @@ public class ControlFlowPointcut implements Pointcut, ClassFilter, MethodMatcher
 
 	private String methodName;
 
-	private int evaluations;
+	private volatile int evaluations;
 
 
 	/**
@@ -55,11 +53,11 @@ public ControlFlowPointcut(Class<?> clazz) {
 	}
 
 	/**
-	 * Construct a new pointcut that matches all calls below the
-	 * given method in the given class. If the method name is null,
-	 * matches all control flows below that class.
+	 * Construct a new pointcut that matches all calls below the given method
+	 * in the given class. If no method name is given, matches all control flows
+	 * below the given class.
 	 * @param clazz the clazz
-	 * @param methodName the name of the method
+	 * @param methodName the name of the method (may be {@code null})
 	 */
 	public ControlFlowPointcut(Class<?> clazz, String methodName) {
 		Assert.notNull(clazz, "Class must not be null");
@@ -93,8 +91,14 @@ public boolean isRuntime() {
 	@Override
 	public boolean matches(Method method, Class<?> targetClass, Object... args) {
 		this.evaluations++;
-		ControlFlow cflow = ControlFlowFactory.createControlFlow();
-		return (this.methodName != null ? cflow.under(this.clazz, this.methodName) : cflow.under(this.clazz));
+
+		for (StackTraceElement element : new Throwable().getStackTrace()) {
+			if (element.getClassName().equals(this.clazz.getName()) &&
+					(this.methodName == null || element.getMethodName().equals(this.methodName))) {
+				return true;
+			}
+		}
+		return false;
 	}
 
 	/**
@@ -130,8 +134,7 @@ public boolean equals(Object other) {
 
 	@Override
 	public int hashCode() {
-		int code = 17;
-		code = 37 * code + this.clazz.hashCode();
+		int code = this.clazz.hashCode();
 		if (this.methodName != null) {
 			code = 37 * code + this.methodName.hashCode();
 		}

spring-beans/src/main/java/org/springframework/beans/PropertyAccessException.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -18,17 +18,15 @@
 
 import java.beans.PropertyChangeEvent;
 
-import org.springframework.core.ErrorCoded;
-
 /**
  * Superclass for exceptions related to a property access,
  * such as type mismatch or invocation target exception.
  *
  * @author Rod Johnson
  * @author Juergen Hoeller
  */
-@SuppressWarnings("serial")
-public abstract class PropertyAccessException extends BeansException implements ErrorCoded {
+@SuppressWarnings({"serial", "deprecation"})
+public abstract class PropertyAccessException extends BeansException implements org.springframework.core.ErrorCoded {
 
 	private transient PropertyChangeEvent propertyChangeEvent;
 
@@ -77,4 +75,10 @@ public Object getValue() {
 		return (this.propertyChangeEvent != null ? this.propertyChangeEvent.getNewValue() : null);
 	}
 
+	/**
+	 * Return a corresponding error code for this type of exception.
+	 */
+	@Override
+	public abstract String getErrorCode();
+
 }

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -23,7 +23,9 @@
  *
  * @author Rod Johnson
  * @since 02.02.2004
+ * @deprecated as of Spring Framework 4.3.6
  */
+@Deprecated
 public interface ControlFlow {
 
 	/**

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -31,7 +31,9 @@
  * @author Rod Johnson
  * @author Juergen Hoeller
  * @since 02.02.2004
+ * @deprecated as of Spring Framework 4.3.6
  */
+@Deprecated
 public abstract class ControlFlowFactory {
 
 	/**
@@ -65,8 +67,8 @@ public Jdk14ControlFlow() {
 		public boolean under(Class<?> clazz) {
 			Assert.notNull(clazz, "Class must not be null");
 			String className = clazz.getName();
-			for (int i = 0; i < stack.length; i++) {
-				if (this.stack[i].getClassName().equals(className)) {
+			for (StackTraceElement element : this.stack) {
+				if (element.getClassName().equals(className)) {
 					return true;
 				}
 			}
@@ -82,9 +84,9 @@ public boolean under(Class<?> clazz, String methodName) {
 			Assert.notNull(clazz, "Class must not be null");
 			Assert.notNull(methodName, "Method name must not be null");
 			String className = clazz.getName();
-			for (int i = 0; i < this.stack.length; i++) {
-				if (this.stack[i].getClassName().equals(className) &&
-						this.stack[i].getMethodName().equals(methodName)) {
+			for (StackTraceElement element : this.stack) {
+				if (element.getClassName().equals(className) &&
+						element.getMethodName().equals(methodName)) {
 					return true;
 				}
 			}
@@ -103,7 +105,7 @@ public boolean underToken(String token) {
 			StringWriter sw = new StringWriter();
 			new Throwable().printStackTrace(new PrintWriter(sw));
 			String stackTrace = sw.toString();
-			return stackTrace.indexOf(token) != -1;
+			return stackTrace.contains(token);
 		}
 
 		@Override

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -25,7 +25,9 @@
  *
  * @author Rod Johnson
  * @see org.springframework.context.MessageSource
+ * @deprecated as of Spring Framework 4.3.6
  */
+@Deprecated
 public interface ErrorCoded {
 
 	/**

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -44,7 +44,9 @@
  * @author Juergen Hoeller
  * @since 1.2
  * @see #monitor
+ * @deprecated as of Spring Framework 4.3.6
  */
+@Deprecated
 public class WeakReferenceMonitor {
 
 	private static final Log logger = LogFactory.getLog(WeakReferenceMonitor.class);

spring-core/src/test/java/org/springframework/core/ControlFlowTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2017 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.
@@ -24,6 +24,7 @@
  * @author Rod Johnson
  * @author Sam Brannen
  */
+@SuppressWarnings("deprecation")
 public class ControlFlowTests {
 
 	@Test
@@ -33,6 +34,7 @@ public void underClassAndMethod() {
 		new Three().test();
 	}
 
+
 	static class One {
 
 		void test() {
@@ -45,6 +47,7 @@ void test() {
 		}
 	}
 
+
 	static class Two {
 
 		void testing() {
@@ -57,6 +60,7 @@ void testing() {
 		}
 	}
 
+
 	static class Three {
 
 		void test() {