Polishing

jhoeller · jhoeller · commit 484addb4f8ca · 2018-07-20T17:04:17.000+02:00
diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.java b/spring-aop/src/main/java/org/springframework/aop/framework/ProxyFactoryBean.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.
@@ -640,7 +640,7 @@ public PrototypePlaceholderAdvisor(String beanName) {
 		}
 
 		public String getBeanName() {
-			return beanName;
+			return this.beanName;
 		}
 
 		@Override
diff --git a/spring-aop/src/main/java/org/springframework/aop/target/AbstractPrototypeBasedTargetSource.java b/spring-aop/src/main/java/org/springframework/aop/target/AbstractPrototypeBasedTargetSource.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2015 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -74,8 +74,8 @@ protected Object newPrototypeInstance() throws BeansException {
 	 * @param target the bean instance to destroy
 	 */
 	protected void destroyPrototypeInstance(Object target) {
-		if (this.logger.isDebugEnabled()) {
-			this.logger.debug("Destroying instance of bean '" + getTargetBeanName() + "'");
+		if (logger.isDebugEnabled()) {
+			logger.debug("Destroying instance of bean '" + getTargetBeanName() + "'");
 		}
 		if (getBeanFactory() instanceof ConfigurableBeanFactory) {
 			((ConfigurableBeanFactory) getBeanFactory()).destroyBean(getTargetBeanName(), target);
diff --git a/spring-context/src/main/java/org/springframework/context/annotation/AutoProxyRegistrar.java b/spring-context/src/main/java/org/springframework/context/annotation/AutoProxyRegistrar.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2016 the original author or authors.
+ * Copyright 2002-2018 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,7 +77,7 @@ public void registerBeanDefinitions(AnnotationMetadata importingClassMetadata, B
 				}
 			}
 		}
-		if (!candidateFound) {
+		if (!candidateFound && logger.isWarnEnabled()) {
 			String name = getClass().getSimpleName();
 			logger.warn(String.format("%s was imported but no annotations were found " +
 					"having both 'mode' and 'proxyTargetClass' attributes of type " +
diff --git a/spring-context/src/main/java/org/springframework/jmx/support/MBeanRegistrationSupport.java b/spring-context/src/main/java/org/springframework/jmx/support/MBeanRegistrationSupport.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -209,7 +209,9 @@ else if (this.registrationPolicy == RegistrationPolicy.REPLACE_EXISTING) {
 						registeredBean = this.server.registerMBean(mbean, objectName);
 					}
 					catch (InstanceNotFoundException ex2) {
-						logger.error("Unable to replace existing MBean at [" + objectName + "]", ex2);
+						if (logger.isErrorEnabled()) {
+							logger.error("Unable to replace existing MBean at [" + objectName + "]", ex2);
+						}
 						throw ex;
 					}
 				}
@@ -239,9 +241,9 @@ protected void unregisterBeans() {
 		}
 		if (!snapshot.isEmpty()) {
 			logger.info("Unregistering JMX-exposed beans");
-		}
-		for (ObjectName objectName : snapshot) {
-			doUnregister(objectName);
+			for (ObjectName objectName : snapshot) {
+				doUnregister(objectName);
+			}
 		}
 	}
 
diff --git a/spring-context/src/main/java/org/springframework/remoting/rmi/RmiServiceExporter.java b/spring-context/src/main/java/org/springframework/remoting/rmi/RmiServiceExporter.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -436,8 +436,8 @@ public void destroy() throws RemoteException {
 		}
 		catch (NotBoundException ex) {
 			if (logger.isWarnEnabled()) {
-				logger.warn("RMI service '" + this.serviceName + "' is not bound to registry"
-						+ (this.createdRegistry ? (" at port '" + this.registryPort + "' anymore") : ""), ex);
+				logger.warn("RMI service '" + this.serviceName + "' is not bound to registry" +
+						(this.createdRegistry ? (" at port '" + this.registryPort + "' anymore") : ""), ex);
 			}
 		}
 		finally {
@@ -454,8 +454,9 @@ private void unexportObjectSilently() {
 		}
 		catch (NoSuchObjectException ex) {
 			if (logger.isWarnEnabled()) {
-				logger.warn("RMI object for service '" + this.serviceName + "' isn't exported anymore", ex);
+				logger.warn("RMI object for service '" + this.serviceName + "' is not exported anymore", ex);
 			}
 		}
 	}
+
 }
diff --git a/spring-context/src/main/java/org/springframework/remoting/support/RemoteInvocationBasedExporter.java b/spring-context/src/main/java/org/springframework/remoting/support/RemoteInvocationBasedExporter.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2007 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -79,13 +79,13 @@ protected Object invoke(RemoteInvocation invocation, Object targetObject)
 		}
 		catch (NoSuchMethodException ex) {
 			if (logger.isDebugEnabled()) {
-				logger.warn("Could not find target method for " + invocation, ex);
+				logger.debug("Could not find target method for " + invocation, ex);
 			}
 			throw ex;
 		}
 		catch (IllegalAccessException ex) {
 			if (logger.isDebugEnabled()) {
-				logger.warn("Could not access target method for " + invocation, ex);
+				logger.debug("Could not access target method for " + invocation, ex);
 			}
 			throw ex;
 		}
diff --git a/spring-core/src/main/java/org/springframework/core/env/AbstractEnvironment.java b/spring-core/src/main/java/org/springframework/core/env/AbstractEnvironment.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -17,6 +17,7 @@
 package org.springframework.core.env;
 
 import java.security.AccessControlException;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.LinkedHashSet;
 import java.util.Map;
@@ -249,6 +250,9 @@ protected Set<String> doGetActiveProfiles() {
 	@Override
 	public void setActiveProfiles(String... profiles) {
 		Assert.notNull(profiles, "Profile array must not be null");
+		if (logger.isDebugEnabled()) {
+			logger.debug("Activating profiles " + Arrays.asList(profiles));
+		}
 		synchronized (this.activeProfiles) {
 			this.activeProfiles.clear();
 			for (String profile : profiles) {
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/CallMetaDataContext.java b/spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/CallMetaDataContext.java
@@ -496,7 +496,7 @@ public Map<String, Object> matchInParameterValuesWithCallParameters(SqlParameter
 										matchedParameters.put(parameterName,
 												SqlParameterSourceUtils.getTypedValue(parameterSource, sourceName));
 									}
-									else {
+									else if (logger.isWarnEnabled()) {
 										logger.warn("Unable to locate the corresponding parameter value for '" +
 												parameterName + "' within the parameter values provided: " +
 												caseInsensitiveParameterNames.values());
@@ -562,7 +562,7 @@ public Map<String, Object> matchInParameterValuesWithCallParameters(SqlParameter
 			for (String parameterName : callParameterNames.keySet()) {
 				String parameterNameToMatch = this.metaDataProvider.parameterNameToUse(parameterName);
 				String callParameterName = callParameterNames.get(parameterNameToMatch.toLowerCase());
-				if (!matchedParameters.containsKey(callParameterName)) {
+				if (!matchedParameters.containsKey(callParameterName) && logger.isWarnEnabled()) {
 					logger.warn("Unable to locate the corresponding parameter value for '" + parameterName +
 							"' within the parameter values provided: " + inParameters.keySet());
 				}
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/TableMetaDataContext.java b/spring-jdbc/src/main/java/org/springframework/jdbc/core/metadata/TableMetaDataContext.java
@@ -182,7 +182,7 @@ protected List<String> reconcileColumnsToUse(List<String> declaredColumns, Strin
 		if (generatedKeyNames.length > 0) {
 			this.generatedKeyColumnsUsed = true;
 		}
-		if (declaredColumns.size() > 0) {
+		if (!declaredColumns.isEmpty()) {
 			return new ArrayList<String>(declaredColumns);
 		}
 		Set<String> keys = new LinkedHashSet<String>(generatedKeyNames.length);
@@ -255,7 +255,7 @@ public List<Object> matchInParameterValuesWithInsertColumns(Map<String, ?> inPar
 
 
 	/**
-	 * Build the insert string based on configuration and meta-data information
+	 * Build the insert string based on configuration and meta-data information.
 	 * @return the insert string to be used
 	 */
 	public String createInsertString(String... generatedKeyNames) {
@@ -284,8 +284,10 @@ public String createInsertString(String... generatedKeyNames) {
 		insertStatement.append(") VALUES(");
 		if (columnCount < 1) {
 			if (this.generatedKeyColumnsUsed) {
-				logger.info("Unable to locate non-key columns for table '" +
-						getTableName() + "' so an empty insert statement is generated");
+				if (logger.isInfoEnabled()) {
+					logger.info("Unable to locate non-key columns for table '" +
+							getTableName() + "' so an empty insert statement is generated");
+				}
 			}
 			else {
 				throw new InvalidDataAccessApiUsageException("Unable to locate columns for table '" +
@@ -352,6 +354,9 @@ public boolean isGetGeneratedKeysSimulated() {
 	}
 
 	/**
+	 * Does this database support a simple query to retrieve generated keys
+	 * when the JDBC 3.0 feature is not supported:
+	 * {@link java.sql.DatabaseMetaData#supportsGetGeneratedKeys()}?
 	 * @deprecated as of 4.3.15, in favor of {@link #getSimpleQueryForGetGeneratedKey}
 	 */
 	@Deprecated
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseFactory.java b/spring-jdbc/src/main/java/org/springframework/jdbc/datasource/embedded/EmbeddedDatabaseFactory.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.
@@ -87,8 +87,8 @@ public class EmbeddedDatabaseFactory {
 	 * generation of a pseudo-random unique ID to be used as the database name.
 	 * <p>Setting this flag to {@code true} overrides any explicit name set
 	 * via {@link #setDatabaseName}.
-	 * @see #setDatabaseName
 	 * @since 4.2
+	 * @see #setDatabaseName
 	 */
 	public void setGenerateUniqueDatabaseName(boolean generateUniqueDatabaseName) {
 		this.generateUniqueDatabaseName = generateUniqueDatabaseName;
@@ -183,7 +183,7 @@ protected void initDatabase() {
 			if (this.dataSource instanceof SimpleDriverDataSource) {
 				SimpleDriverDataSource simpleDriverDataSource = (SimpleDriverDataSource) this.dataSource;
 				logger.info(String.format("Starting embedded database: url='%s', username='%s'",
-					simpleDriverDataSource.getUrl(), simpleDriverDataSource.getUsername()));
+						simpleDriverDataSource.getUrl(), simpleDriverDataSource.getUsername()));
 			}
 			else {
 				logger.info(String.format("Starting embedded database '%s'", this.databaseName));
@@ -211,7 +211,6 @@ protected void initDatabase() {
 	 */
 	protected void shutdownDatabase() {
 		if (this.dataSource != null) {
-
 			if (logger.isInfoEnabled()) {
 				if (this.dataSource instanceof SimpleDriverDataSource) {
 					logger.info(String.format("Shutting down embedded database: url='%s'",
@@ -221,7 +220,6 @@ protected void shutdownDatabase() {
 					logger.info(String.format("Shutting down embedded database '%s'", this.databaseName));
 				}
 			}
-
 			this.databaseConfigurer.shutdown(this.dataSource, this.databaseName);
 			this.dataSource = null;
 		}
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/support/SQLErrorCodesFactory.java b/spring-jdbc/src/main/java/org/springframework/jdbc/support/SQLErrorCodesFactory.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -123,8 +123,8 @@ protected SQLErrorCodesFactory() {
 
 			// Check all beans of type SQLErrorCodes.
 			errorCodes = lbf.getBeansOfType(SQLErrorCodes.class, true, false);
-			if (logger.isInfoEnabled()) {
-				logger.info("SQLErrorCodes loaded: " + errorCodes.keySet());
+			if (logger.isDebugEnabled()) {
+				logger.debug("SQLErrorCodes loaded: " + errorCodes.keySet());
 			}
 		}
 		catch (BeansException ex) {
diff --git a/spring-jdbc/src/main/java/org/springframework/jdbc/support/lob/TemporaryLobCreator.java b/spring-jdbc/src/main/java/org/springframework/jdbc/support/lob/TemporaryLobCreator.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -187,7 +187,7 @@ public void close() {
 			}
 		}
 		catch (SQLException ex) {
-			logger.error("Could not free LOB", ex);
+			logger.error("Could not free LOBs", ex);
 		}
 	}
 
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/support/AbstractHeaderMapper.java b/spring-messaging/src/main/java/org/springframework/messaging/support/AbstractHeaderMapper.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2014 the original author or authors.
+ * Copyright 2002-2018 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,7 @@
 import org.springframework.util.StringUtils;
 
 /**
- * A base {@link HeaderMapper} implementation
+ * A base {@link HeaderMapper} implementation.
  *
  * @author Stephane Nicoll
  * @since 4.1
diff --git a/spring-oxm/src/main/java/org/springframework/oxm/jibx/JibxMarshaller.java b/spring-oxm/src/main/java/org/springframework/oxm/jibx/JibxMarshaller.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2016 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -208,7 +208,8 @@ public void afterPropertiesSet() throws JiBXException {
 		if (this.targetClass != null) {
 			if (StringUtils.hasLength(this.bindingName)) {
 				if (logger.isInfoEnabled()) {
-					logger.info("Configured for target class [" + this.targetClass + "] using binding [" + this.bindingName + "]");
+					logger.info("Configured for target class [" + this.targetClass +
+							"] using binding [" + this.bindingName + "]");
 				}
 				this.bindingFactory = BindingDirectory.getFactory(this.bindingName, this.targetClass);
 			}
@@ -220,11 +221,12 @@ public void afterPropertiesSet() throws JiBXException {
 			}
 		}
 		else if (this.targetPackage != null) {
-			if (!StringUtils.hasLength(bindingName)) {
-				bindingName = DEFAULT_BINDING_NAME;
+			if (!StringUtils.hasLength(this.bindingName)) {
+				this.bindingName = DEFAULT_BINDING_NAME;
 			}
 			if (logger.isInfoEnabled()) {
-				logger.info("Configured for target package [" + this.targetPackage	+ "] using binding [" + this.bindingName + "]");
+				logger.info("Configured for target package [" + this.targetPackage +
+						"] using binding [" + this.bindingName + "]");
 			}
 			this.bindingFactory = BindingDirectory.getFactory(this.bindingName, this.targetPackage);
 		}
@@ -279,9 +281,10 @@ protected void marshalWriter(Object graph, Writer writer) throws XmlMappingExcep
 	}
 
 	private void marshalDocument(IMarshallingContext marshallingContext, Object graph) throws IOException, JiBXException {
-		if (StringUtils.hasLength(docTypeRootElementName)) {
+		if (StringUtils.hasLength(this.docTypeRootElementName)) {
 			IXMLWriter xmlWriter = marshallingContext.getXmlWriter();
-			xmlWriter.writeDocType(docTypeRootElementName, docTypeSystemId, docTypePublicId, docTypeInternalSubset);
+			xmlWriter.writeDocType(this.docTypeRootElementName, this.docTypeSystemId,
+					this.docTypePublicId, this.docTypeInternalSubset);
 		}
 		marshallingContext.marshalDocument(graph);
 	}
@@ -380,7 +383,7 @@ protected Object unmarshalXmlStreamReader(XMLStreamReader streamReader) {
 	protected Object unmarshalInputStream(InputStream inputStream) throws XmlMappingException, IOException {
 		try {
 			IUnmarshallingContext unmarshallingContext = createUnmarshallingContext();
-			return unmarshallingContext.unmarshalDocument(inputStream, encoding);
+			return unmarshallingContext.unmarshalDocument(inputStream, this.encoding);
 		}
 		catch (JiBXException ex) {
 			throw convertJibxException(ex, false);
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketAnnotationMethodMessageHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketAnnotationMethodMessageHandler.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2018 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.
@@ -74,7 +74,9 @@ private void initMessagingAdviceCache(List<MessagingAdviceBean> beans) {
 			AnnotationExceptionHandlerMethodResolver resolver = new AnnotationExceptionHandlerMethodResolver(type);
 			if (resolver.hasExceptionMappings()) {
 				registerExceptionHandlerAdvice(bean, resolver);
-				logger.info("Detected @MessageExceptionHandler methods in " + bean);
+				if (logger.isInfoEnabled()) {
+					logger.info("Detected @MessageExceptionHandler methods in " + bean);
+				}
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright 2002-2015 the original author or authors.`
	`2`	`+ * Copyright 2002-2017 the original author or authors.`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -640,7 +640,7 @@ public PrototypePlaceholderAdvisor(String beanName) {`
`640`	`640`	`}`
`641`	`641`
`642`	`642`	`public String getBeanName() {`
`643`		`- return beanName;`
	`643`	`+ return this.beanName;`
`644`	`644`	`}`
`645`	`645`
`646`	`646`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright 2002-2016 the original author or authors.`
	`2`	`+ * Copyright 2002-2018 the original author or authors.`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -77,7 +77,7 @@ public void registerBeanDefinitions(AnnotationMetadata importingClassMetadata, B`
`77`	`77`	`}`
`78`	`78`	`}`
`79`	`79`	`}`
`80`		`- if (!candidateFound) {`
	`80`	`+ if (!candidateFound && logger.isWarnEnabled()) {`
`81`	`81`	`String name = getClass().getSimpleName();`
`82`	`82`	`logger.warn(String.format("%s was imported but no annotations were found " +`
`83`	`83`	`"having both 'mode' and 'proxyTargetClass' attributes of type " +`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright 2002-2014 the original author or authors.`
	`2`	`+ * Copyright 2002-2018 the original author or authors.`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -209,7 +209,9 @@ else if (this.registrationPolicy == RegistrationPolicy.REPLACE_EXISTING) {`
`209`	`209`	`registeredBean = this.server.registerMBean(mbean, objectName);`
`210`	`210`	`}`
`211`	`211`	`catch (InstanceNotFoundException ex2) {`
`212`		`- logger.error("Unable to replace existing MBean at [" + objectName + "]", ex2);`
	`212`	`+ if (logger.isErrorEnabled()) {`
	`213`	`+ logger.error("Unable to replace existing MBean at [" + objectName + "]", ex2);`
	`214`	`+ }`
`213`	`215`	`throw ex;`
`214`	`216`	`}`
`215`	`217`	`}`
`@@ -239,9 +241,9 @@ protected void unregisterBeans() {`
`239`	`241`	`}`
`240`	`242`	`if (!snapshot.isEmpty()) {`
`241`	`243`	`logger.info("Unregistering JMX-exposed beans");`
`242`		`- }`
`243`		`- for (ObjectName objectName : snapshot) {`
`244`		`- doUnregister(objectName);`
	`244`	`+ for (ObjectName objectName : snapshot) {`
	`245`	`+ doUnregister(objectName);`
	`246`	`+ }`
`245`	`247`	`}`
`246`	`248`	`}`
`247`	`249`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright 2002-2012 the original author or authors.`
	`2`	`+ * Copyright 2002-2018 the original author or authors.`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -436,8 +436,8 @@ public void destroy() throws RemoteException {`
`436`	`436`	`}`
`437`	`437`	`catch (NotBoundException ex) {`
`438`	`438`	`if (logger.isWarnEnabled()) {`
`439`		`- logger.warn("RMI service '" + this.serviceName + "' is not bound to registry"`
`440`		`- + (this.createdRegistry ? (" at port '" + this.registryPort + "' anymore") : ""), ex);`
	`439`	`+ logger.warn("RMI service '" + this.serviceName + "' is not bound to registry" +`
	`440`	`+ (this.createdRegistry ? (" at port '" + this.registryPort + "' anymore") : ""), ex);`
`441`	`441`	`}`
`442`	`442`	`}`
`443`	`443`	`finally {`
`@@ -454,8 +454,9 @@ private void unexportObjectSilently() {`
`454`	`454`	`}`
`455`	`455`	`catch (NoSuchObjectException ex) {`
`456`	`456`	`if (logger.isWarnEnabled()) {`
`457`		`- logger.warn("RMI object for service '" + this.serviceName + "' isn't exported anymore", ex);`
	`457`	`+ logger.warn("RMI object for service '" + this.serviceName + "' is not exported anymore", ex);`
`458`	`458`	`}`
`459`	`459`	`}`
`460`	`460`	`}`
	`461`	`+`
`461`	`462`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/*`
`2`		`- * Copyright 2002-2007 the original author or authors.`
	`2`	`+ * Copyright 2002-2018 the original author or authors.`
`3`	`3`	`*`
`4`	`4`	`* Licensed under the Apache License, Version 2.0 (the "License");`
`5`	`5`	`* you may not use this file except in compliance with the License.`
`@@ -79,13 +79,13 @@ protected Object invoke(RemoteInvocation invocation, Object targetObject)`
`79`	`79`	`}`
`80`	`80`	`catch (NoSuchMethodException ex) {`
`81`	`81`	`if (logger.isDebugEnabled()) {`
`82`		`- logger.warn("Could not find target method for " + invocation, ex);`
	`82`	`+ logger.debug("Could not find target method for " + invocation, ex);`
`83`	`83`	`}`
`84`	`84`	`throw ex;`
`85`	`85`	`}`
`86`	`86`	`catch (IllegalAccessException ex) {`
`87`	`87`	`if (logger.isDebugEnabled()) {`
`88`		`- logger.warn("Could not access target method for " + invocation, ex);`
	`88`	`+ logger.debug("Could not access target method for " + invocation, ex);`
`89`	`89`	`}`
`90`	`90`	`throw ex;`
`91`	`91`	`}`