fix ignored exceptions

Vincent Potucek · Vincent Potucek · commit e6c4c5fd509d · 2025-06-13T16:10:54.000+02:00
Signed-off-by: Vincent Potucek &lt;vpotucek@me.com&gt;
diff --git a/integration-tests/src/test/java/org/springframework/aop/framework/autoproxy/AdvisorAutoProxyCreatorIntegrationTests.java b/integration-tests/src/test/java/org/springframework/aop/framework/autoproxy/AdvisorAutoProxyCreatorIntegrationTests.java
@@ -158,8 +158,7 @@ void testRollbackRulesOnMethodPreventRollback() throws Exception {
 		try {
 			rb.echoException(new ServletException());
 		}
-		catch (ServletException ex) {
-
+		catch (ServletException ignored) {
 		}
 		assertThat(txMan.commits).as("Transaction counts match").isEqualTo(1);
 	}
@@ -272,7 +271,7 @@ public void before(Method method, Object[] args, Object target) throws Throwable
 					TransactionInterceptor.currentTransactionStatus();
 					throw new RuntimeException("Shouldn't have a transaction");
 				}
-				catch (NoTransactionException ex) {
+				catch (NoTransactionException ignored) {
 					// this is Ok
 				}
 			}
diff --git a/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJExpressionPointcut.java b/spring-aop/src/main/java/org/springframework/aop/aspectj/AspectJExpressionPointcut.java
@@ -453,9 +453,9 @@ private ShadowMatch getTargetShadowMatch(Method method, Class<?> targetClass) {
 							ClassUtils.toClassArray(ifcs), targetClass.getClassLoader());
 					targetMethod = ClassUtils.getMostSpecificMethod(targetMethod, compositeInterface);
 				}
-				catch (IllegalArgumentException ex) {
-					// Implemented interfaces probably expose conflicting method signatures...
-					// Proceed with original target method.
+				// Implemented interfaces probably expose conflicting method signatures...
+				// Proceed with original target method.
+				catch (IllegalArgumentException ignored) {
 				}
 			}
 		}
@@ -478,7 +478,7 @@ private ShadowMatch getShadowMatch(Method targetMethod, Method originalMethod) {
 					try {
 						shadowMatch = pointcutExpression.matchesMethodExecution(methodToMatch);
 					}
-					catch (ReflectionWorldException ex) {
+					catch (ReflectionWorldException ignored) {
 						// Failed to introspect target method, probably because it has been loaded
 						// in a special ClassLoader. Let's try the declaring ClassLoader instead...
 						try {
@@ -501,7 +501,7 @@ private ShadowMatch getShadowMatch(Method targetMethod, Method originalMethod) {
 						try {
 							shadowMatch = pointcutExpression.matchesMethodExecution(methodToMatch);
 						}
-						catch (ReflectionWorldException ex) {
+						catch (ReflectionWorldException ignored) {
 							// Could neither introspect the target class nor the proxy class ->
 							// let's try the original method's declaring class before we give up...
 							try {
diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/annotation/BeanFactoryAnnotationUtils.java b/spring-beans/src/main/java/org/springframework/beans/factory/annotation/BeanFactoryAnnotationUtils.java
@@ -208,8 +208,8 @@ public static boolean isQualifierMatch(
 					}
 				}
 			}
-			catch (NoSuchBeanDefinitionException ex) {
-				// Ignore - can't compare qualifiers for a manually registered singleton object
+			catch (NoSuchBeanDefinitionException ignored) {
+				// can't compare qualifiers for a manually registered singleton object
 			}
 		}
 		return false;
diff --git a/spring-context/src/main/java/org/springframework/context/support/PostProcessorRegistrationDelegate.java b/spring-context/src/main/java/org/springframework/context/support/PostProcessorRegistrationDelegate.java
@@ -523,8 +523,7 @@ private void resolveTypeStringValue(TypedStringValue typedStringValue) {
 			try {
 				typedStringValue.resolveTargetType(this.beanFactory.getBeanClassLoader());
 			}
-			catch (ClassNotFoundException ex) {
-				// ignore
+			catch (ClassNotFoundException ignored) {
 			}
 		}
 
diff --git a/spring-context/src/main/java/org/springframework/validation/beanvalidation/LocalValidatorFactoryBean.java b/spring-context/src/main/java/org/springframework/validation/beanvalidation/LocalValidatorFactoryBean.java
@@ -266,8 +266,8 @@ public void afterPropertiesSet() {
 				Method eclMethod = configuration.getClass().getMethod("externalClassLoader", ClassLoader.class);
 				ReflectionUtils.invokeMethod(eclMethod, configuration, this.applicationContext.getClassLoader());
 			}
-			catch (NoSuchMethodException ex) {
-				// Ignore - no Hibernate Validator 5.2+ or similar provider
+			catch (NoSuchMethodException ignored) {
+				// no Hibernate Validator 5.2+ or similar provider
 			}
 		}
 
@@ -417,8 +417,8 @@ public <T> T unwrap(@Nullable Class<T> type) {
 			try {
 				return super.unwrap(type);
 			}
-			catch (ValidationException ex) {
-				// Ignore - we'll try ValidatorFactory unwrapping next
+			catch (ValidationException ignored) {
+				// we'll try ValidatorFactory unwrapping next
 			}
 		}
 		if (this.validatorFactory != null) {
diff --git a/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java b/spring-context/src/test/java/org/springframework/aop/framework/AbstractAopProxyTests.java
@@ -233,8 +233,7 @@ void serializableTargetAndAdvice() throws Throwable {
 		try {
 			p2.echo(new IOException());
 		}
-		catch (IOException ex) {
-
+		catch (IOException ignored) {
 		}
 		assertThat(cta.getCalls()).isEqualTo(2);
 	}
diff --git a/spring-core-test/src/main/java/org/springframework/core/test/tools/CompileWithForkedClassLoaderClassLoader.java b/spring-core-test/src/main/java/org/springframework/core/test/tools/CompileWithForkedClassLoaderClassLoader.java
@@ -84,8 +84,7 @@ protected Class<?> findClass(String name) throws ClassNotFoundException {
 			try (stream) {
 				return stream.readAllBytes();
 			}
-			catch (IOException ex) {
-				// ignore
+			catch (IOException ignored) {
 			}
 		}
 		return null;
diff --git a/spring-core/src/main/java/org/springframework/cglib/proxy/BridgeMethodResolver.java b/spring-core/src/main/java/org/springframework/cglib/proxy/BridgeMethodResolver.java
@@ -73,7 +73,8 @@ public BridgeMethodResolver(Map declToBridge, ClassLoader classLoader) {
                 } finally {
                     is.close();
                 }
-            } catch (IOException ignored) {}
+            } catch (IOException ignored) {
+			}
         }
         return resolved;
     }
diff --git a/spring-core/src/main/java/org/springframework/cglib/proxy/MethodProxy.java b/spring-core/src/main/java/org/springframework/cglib/proxy/MethodProxy.java
@@ -62,8 +62,8 @@ public static MethodProxy create(Class c1, Class c2, String desc, String name1,
 			try {
 				proxy.init();
 			}
-			catch (CodeGenerationException ex) {
-				// Ignore - to be retried when actually needed later on (possibly not at all)
+			catch (CodeGenerationException ignored) {
+				// to be retried when actually needed later on (possibly not at all)
 			}
 		}
 		// SPRING PATCH END
diff --git a/spring-core/src/main/java/org/springframework/core/io/support/SpringFactoriesLoader.java b/spring-core/src/main/java/org/springframework/core/io/support/SpringFactoriesLoader.java
@@ -438,8 +438,7 @@ private static class KotlinDelegate {
 					return constructor;
 				}
 			}
-			catch (UnsupportedOperationException ex) {
-				// ignore
+			catch (UnsupportedOperationException ignored) {
 			}
 			return null;
 		}
diff --git a/spring-core/src/main/java/org/springframework/util/FileCopyUtils.java b/spring-core/src/main/java/org/springframework/util/FileCopyUtils.java
@@ -229,8 +229,7 @@ private static void close(Closeable closeable) {
 		try {
 			closeable.close();
 		}
-		catch (IOException ex) {
-			// ignore
+		catch (IOException ignored) {
 		}
 	}
 
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/DefaultStompSession.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/DefaultStompSession.java
@@ -526,8 +526,7 @@ private void resetConnection() {
 			try {
 				conn.close();
 			}
-			catch (Throwable ex) {
-				// ignore
+			catch (Throwable ignored) {
 			}
 		}
 	}
diff --git a/spring-oxm/src/main/java/org/springframework/oxm/jaxb/Jaxb2Marshaller.java b/spring-oxm/src/main/java/org/springframework/oxm/jaxb/Jaxb2Marshaller.java
@@ -1000,8 +1000,7 @@ private String getHost(String elementNamespace, DataHandler dataHandler) {
 				URI uri = ResourceUtils.toURI(elementNamespace);
 				return uri.getHost();
 			}
-			catch (URISyntaxException ex) {
-				// ignore
+			catch (URISyntaxException ignored) {
 			}
 			return dataHandler.getName();
 		}
diff --git a/spring-r2dbc/src/test/java/org/springframework/r2dbc/connection/R2dbcTransactionManagerTests.java b/spring-r2dbc/src/test/java/org/springframework/r2dbc/connection/R2dbcTransactionManagerTests.java
@@ -710,8 +710,7 @@ public Mono<Void> afterCompletion(int status) {
 			try {
 				return Mono.fromRunnable(() -> doAfterCompletion(status));
 			}
-			catch (Throwable ex) {
-				// ignore
+			catch (Throwable ignored) {
 			}
 
 			return Mono.empty();
diff --git a/spring-test/src/main/java/org/springframework/mock/http/client/MockClientHttpResponse.java b/spring-test/src/main/java/org/springframework/mock/http/client/MockClientHttpResponse.java
@@ -100,8 +100,7 @@ public void close() {
 		try {
 			getBody().close();
 		}
-		catch (IOException ex) {
-			// ignore
+		catch (IOException ignored) {
 		}
 	}
 
diff --git a/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletRequest.java b/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletRequest.java
@@ -1129,8 +1129,7 @@ private long parseDateHeader(String name, String value) {
 			try {
 				return simpleDateFormat.parse(value).getTime();
 			}
-			catch (ParseException ex) {
-				// ignore
+			catch (ParseException ignored) {
 			}
 		}
 		throw new IllegalArgumentException("Cannot parse date value '" + value + "' for '" + name + "' header");
diff --git a/spring-web/src/main/java/org/springframework/http/HttpHeaders.java b/spring-web/src/main/java/org/springframework/http/HttpHeaders.java
@@ -1223,8 +1223,7 @@ public void setHost(@Nullable InetSocketAddress host) {
 			try {
 				port = Integer.parseInt(portString);
 			}
-			catch (NumberFormatException ex) {
-				// ignore
+			catch (NumberFormatException ignored) {
 			}
 		}
 
diff --git a/spring-web/src/main/java/org/springframework/http/client/HttpComponentsClientHttpResponse.java b/spring-web/src/main/java/org/springframework/http/client/HttpComponentsClientHttpResponse.java
@@ -89,8 +89,7 @@ public void close() {
 				this.httpResponse.close();
 			}
 		}
-		catch (IOException ex) {
-			// Ignore exception on close...
+		catch (IOException ignored) {
 		}
 	}
 
diff --git a/spring-web/src/main/java/org/springframework/http/client/ReactorClientHttpResponse.java b/spring-web/src/main/java/org/springframework/http/client/ReactorClientHttpResponse.java
@@ -108,8 +108,7 @@ public void close() {
 			StreamUtils.drain(body);
 			body.close();
 		}
-		catch (IOException ex) {
-			// ignore
+		catch (IOException ignored) {
 		}
 	}
 
diff --git a/spring-web/src/main/java/org/springframework/http/client/reactive/HttpComponentsClientHttpResponse.java b/spring-web/src/main/java/org/springframework/http/client/reactive/HttpComponentsClientHttpResponse.java
@@ -119,8 +119,7 @@ else if (expiresAttribute != null) {
 				ZonedDateTime expiresDate = ZonedDateTime.parse(expiresAttribute, DateTimeFormatter.RFC_1123_DATE_TIME);
 				return Duration.between(ZonedDateTime.now(expiresDate.getZone()), expiresDate).toSeconds();
 			}
-			catch (DateTimeParseException ex) {
-				// ignore
+			catch (DateTimeParseException ignored) {
 			}
 		}
 		return -1;
diff --git a/spring-web/src/main/java/org/springframework/http/converter/ResourceHttpMessageConverter.java b/spring-web/src/main/java/org/springframework/http/converter/ResourceHttpMessageConverter.java
@@ -158,20 +158,20 @@ protected void writeContent(Resource resource, HttpOutputMessage outputMessage)
 				in.transferTo(out);
 				out.flush();
 			}
-			catch (NullPointerException ex) {
-				// ignore, see SPR-13620
+			catch (NullPointerException ignored) {
+				// see SPR-13620
 			}
 			finally {
 				try {
 					in.close();
 				}
-				catch (Throwable ex) {
-					// ignore, see SPR-12999
+				catch (Throwable ignored) {
+					// see SPR-12999
 				}
 			}
 		}
-		catch (FileNotFoundException ex) {
-			// ignore, see SPR-12999
+		catch (FileNotFoundException ignored) {
+			// see SPR-12999
 		}
 	}
 
diff --git a/spring-web/src/main/java/org/springframework/http/converter/ResourceRegionHttpMessageConverter.java b/spring-web/src/main/java/org/springframework/http/converter/ResourceRegionHttpMessageConverter.java
@@ -183,8 +183,7 @@ protected void writeResourceRegion(ResourceRegion region, HttpOutputMessage outp
 			try {
 				in.close();
 			}
-			catch (IOException ex) {
-				// ignore
+			catch (IOException ignored) {
 			}
 		}
 	}
@@ -244,8 +243,7 @@ private void writeResourceRegionCollection(Collection<ResourceRegion> resourceRe
 					in.close();
 				}
 			}
-			catch (IOException ex) {
-				// ignore
+			catch (IOException ignored) {
 			}
 		}
 
diff --git a/spring-web/src/main/java/org/springframework/http/converter/xml/SourceHttpMessageConverter.java b/spring-web/src/main/java/org/springframework/http/converter/xml/SourceHttpMessageConverter.java
@@ -270,8 +270,7 @@ private StreamSource readStreamSource(InputStream body) throws IOException {
 				transform(t, new StreamResult(os));
 				return os.count;
 			}
-			catch (TransformerException ex) {
-				// ignore
+			catch (TransformerException ignored) {
 			}
 		}
 		return null;
diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java b/spring-web/src/main/java/org/springframework/http/server/reactive/AbstractListenerWriteProcessor.java
@@ -439,8 +439,7 @@ public <T> void onNext(AbstractListenerWriteProcessor<T> processor, T data) {
 				// ignore
 			}
 			@Override
-			public <T> void onError(AbstractListenerWriteProcessor<T> processor, Throwable ex) {
-				// ignore
+			public <T> void onError(AbstractListenerWriteProcessor<T> processor, Throwable ignored) {
 			}
 			@Override
 			public <T> void onComplete(AbstractListenerWriteProcessor<T> processor) {
diff --git a/spring-web/src/main/java/org/springframework/http/server/reactive/ServletHttpHandlerAdapter.java b/spring-web/src/main/java/org/springframework/http/server/reactive/ServletHttpHandlerAdapter.java
@@ -309,26 +309,23 @@ private static void delegateTimeout(AsyncListener listener, AsyncEvent event) {
 			try {
 				listener.onTimeout(event);
 			}
-			catch (Exception ex) {
-				// Ignore
+			catch (Exception ignored) {
 			}
 		}
 
 		private static void delegateError(AsyncListener listener, AsyncEvent event) {
 			try {
 				listener.onError(event);
 			}
-			catch (Exception ex) {
-				// Ignore
+			catch (Exception ignored) {
 			}
 		}
 
 		private static void delegateComplete(AsyncListener listener, AsyncEvent event) {
 			try {
 				listener.onComplete(event);
 			}
-			catch (Exception ex) {
-				// Ignore
+			catch (Exception ignored) {
 			}
 		}
 
diff --git a/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java b/spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java
@@ -152,8 +152,7 @@ private static byte[] getResponseBody(ClientHttpResponse response) {
 		try {
 			return FileCopyUtils.copyToByteArray(response.getBody());
 		}
-		catch (IOException ex) {
-			// ignore
+		catch (IOException ignored) {
 		}
 		return new byte[0];
 	}
diff --git a/spring-web/src/main/java/org/springframework/web/context/request/async/CallableInterceptorChain.java b/spring-web/src/main/java/org/springframework/web/context/request/async/CallableInterceptorChain.java
@@ -113,8 +113,7 @@ private void cancelTask() {
 			try {
 				future.cancel(true);
 			}
-			catch (Throwable ex) {
-				// Ignore
+			catch (Throwable ignored) {
 			}
 		}
 	}
diff --git a/spring-web/src/main/java/org/springframework/web/context/request/async/StandardServletAsyncWebRequest.java b/spring-web/src/main/java/org/springframework/web/context/request/async/StandardServletAsyncWebRequest.java
@@ -235,8 +235,7 @@ private int tryObtainLock() {
 					break;
 				}
 			}
-			catch (InterruptedException ex) {
-				// ignore
+			catch (InterruptedException ignored) {
 			}
 		}
 
diff --git a/spring-web/src/main/java/org/springframework/web/context/support/ServletContextResource.java b/spring-web/src/main/java/org/springframework/web/context/support/ServletContextResource.java
@@ -122,8 +122,7 @@ public boolean isReadable() {
 			try {
 				is.close();
 			}
-			catch (IOException ex) {
-				// ignore
+			catch (IOException ignored) {
 			}
 			return true;
 		}
diff --git a/spring-web/src/main/java/org/springframework/web/service/registry/HttpServiceProxyRegistryFactoryBean.java b/spring-web/src/main/java/org/springframework/web/service/registry/HttpServiceProxyRegistryFactoryBean.java
@@ -151,8 +151,7 @@ private static void addGroupAdapter(
 				Class<?> clazz = ClassUtils.forName(className, HttpServiceGroupAdapter.class.getClassLoader());
 				groupAdapters.put(clientType, (HttpServiceGroupAdapter<?>) BeanUtils.instantiateClass(clazz));
 			}
-			catch (ClassNotFoundException ex) {
-				// ignore
+			catch (ClassNotFoundException ignored) {
 			}
 		}
 	}
diff --git a/spring-web/src/test/java/org/springframework/web/context/request/RequestContextListenerTests.java b/spring-web/src/test/java/org/springframework/web/context/request/RequestContextListenerTests.java
@@ -93,7 +93,7 @@ void requestContextListenerWithDifferentThread() {
 		try {
 			thread.join();
 		}
-		catch (InterruptedException ex) {
+		catch (InterruptedException ignored) {
 		}
 		// Still bound to original thread, but at least completed.
 		assertThat(RequestContextHolder.getRequestAttributes()).isNotNull();
diff --git a/spring-web/src/testFixtures/java/org/springframework/web/testfixture/http/client/MockClientHttpResponse.java b/spring-web/src/testFixtures/java/org/springframework/web/testfixture/http/client/MockClientHttpResponse.java
diff --git a/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/MockHttpServletRequest.java b/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/MockHttpServletRequest.java
diff --git a/spring-webflux/src/main/java/org/springframework/web/reactive/function/client/WebClientUtils.java b/spring-webflux/src/main/java/org/springframework/web/reactive/function/client/WebClientUtils.java
diff --git a/spring-webflux/src/main/java/org/springframework/web/reactive/function/server/DefaultServerRequestBuilder.java b/spring-webflux/src/main/java/org/springframework/web/reactive/function/server/DefaultServerRequestBuilder.java
diff --git a/spring-webflux/src/main/java/org/springframework/web/reactive/resource/ResourceHandlerUtils.java b/spring-webflux/src/main/java/org/springframework/web/reactive/resource/ResourceHandlerUtils.java
diff --git a/spring-webflux/src/main/java/org/springframework/web/reactive/result/condition/ProducesRequestCondition.java b/spring-webflux/src/main/java/org/springframework/web/reactive/result/condition/ProducesRequestCondition.java
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/MvcUriComponentsBuilder.java
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartMethodArgumentResolver.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestPartMethodArgumentResolver.java
diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/ResourceHandlerUtils.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/resource/ResourceHandlerUtils.java
diff --git a/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/ExceptionHandlerExceptionResolverTests.java b/spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/ExceptionHandlerExceptionResolverTests.java
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/handler/BinaryWebSocketHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/handler/BinaryWebSocketHandler.java
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/handler/ExceptionWebSocketHandlerDecorator.java b/spring-websocket/src/main/java/org/springframework/web/socket/handler/ExceptionWebSocketHandlerDecorator.java
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/handler/TextWebSocketHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/handler/TextWebSocketHandler.java
diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/messaging/StompSubProtocolHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/messaging/StompSubProtocolHandler.java

Original file line number	Diff line number	Diff line change
`@@ -158,8 +158,7 @@ void testRollbackRulesOnMethodPreventRollback() throws Exception {`
`158`	`158`	`try {`
`159`	`159`	`rb.echoException(new ServletException());`
`160`	`160`	`}`
`161`		`- catch (ServletException ex) {`
`162`		`-`
	`161`	`+ catch (ServletException ignored) {`
`163`	`162`	`}`
`164`	`163`	`assertThat(txMan.commits).as("Transaction counts match").isEqualTo(1);`
`165`	`164`	`}`
`@@ -272,7 +271,7 @@ public void before(Method method, Object[] args, Object target) throws Throwable`
`272`	`271`	`TransactionInterceptor.currentTransactionStatus();`
`273`	`272`	`throw new RuntimeException("Shouldn't have a transaction");`
`274`	`273`	`}`
`275`		`- catch (NoTransactionException ex) {`
	`274`	`+ catch (NoTransactionException ignored) {`
`276`	`275`	`// this is Ok`
`277`	`276`	`}`
`278`	`277`	`}`
Original file line number	Diff line number	Diff line change
`@@ -453,9 +453,9 @@ private ShadowMatch getTargetShadowMatch(Method method, Class<?> targetClass) {`
`453`	`453`	`ClassUtils.toClassArray(ifcs), targetClass.getClassLoader());`
`454`	`454`	`targetMethod = ClassUtils.getMostSpecificMethod(targetMethod, compositeInterface);`
`455`	`455`	`}`
`456`		`- catch (IllegalArgumentException ex) {`
`457`		`- // Implemented interfaces probably expose conflicting method signatures...`
`458`		`- // Proceed with original target method.`
	`456`	`+ // Implemented interfaces probably expose conflicting method signatures...`
	`457`	`+ // Proceed with original target method.`
	`458`	`+ catch (IllegalArgumentException ignored) {`
`459`	`459`	`}`
`460`	`460`	`}`
`461`	`461`	`}`
`@@ -478,7 +478,7 @@ private ShadowMatch getShadowMatch(Method targetMethod, Method originalMethod) {`
`478`	`478`	`try {`
`479`	`479`	`shadowMatch = pointcutExpression.matchesMethodExecution(methodToMatch);`
`480`	`480`	`}`
`481`		`- catch (ReflectionWorldException ex) {`
	`481`	`+ catch (ReflectionWorldException ignored) {`
`482`	`482`	`// Failed to introspect target method, probably because it has been loaded`
`483`	`483`	`// in a special ClassLoader. Let's try the declaring ClassLoader instead...`
`484`	`484`	`try {`
`@@ -501,7 +501,7 @@ private ShadowMatch getShadowMatch(Method targetMethod, Method originalMethod) {`
`501`	`501`	`try {`
`502`	`502`	`shadowMatch = pointcutExpression.matchesMethodExecution(methodToMatch);`
`503`	`503`	`}`
`504`		`- catch (ReflectionWorldException ex) {`
	`504`	`+ catch (ReflectionWorldException ignored) {`
`505`	`505`	`// Could neither introspect the target class nor the proxy class ->`
`506`	`506`	`// let's try the original method's declaring class before we give up...`
`507`	`507`	`try {`
Original file line number	Diff line number	Diff line change
`@@ -208,8 +208,8 @@ public static boolean isQualifierMatch(`
`208`	`208`	`}`
`209`	`209`	`}`
`210`	`210`	`}`
`211`		`- catch (NoSuchBeanDefinitionException ex) {`
`212`		`- // Ignore - can't compare qualifiers for a manually registered singleton object`
	`211`	`+ catch (NoSuchBeanDefinitionException ignored) {`
	`212`	`+ // can't compare qualifiers for a manually registered singleton object`
`213`	`213`	`}`
`214`	`214`	`}`
`215`	`215`	`return false;`
Original file line number	Diff line number	Diff line change
`@@ -523,8 +523,7 @@ private void resolveTypeStringValue(TypedStringValue typedStringValue) {`
`523`	`523`	`try {`
`524`	`524`	`typedStringValue.resolveTargetType(this.beanFactory.getBeanClassLoader());`
`525`	`525`	`}`
`526`		`- catch (ClassNotFoundException ex) {`
`527`		`- // ignore`
	`526`	`+ catch (ClassNotFoundException ignored) {`
`528`	`527`	`}`
`529`	`528`	`}`
`530`	`529`
Original file line number	Diff line number	Diff line change
`@@ -266,8 +266,8 @@ public void afterPropertiesSet() {`
`266`	`266`	`Method eclMethod = configuration.getClass().getMethod("externalClassLoader", ClassLoader.class);`
`267`	`267`	`ReflectionUtils.invokeMethod(eclMethod, configuration, this.applicationContext.getClassLoader());`
`268`	`268`	`}`
`269`		`- catch (NoSuchMethodException ex) {`
`270`		`- // Ignore - no Hibernate Validator 5.2+ or similar provider`
	`269`	`+ catch (NoSuchMethodException ignored) {`
	`270`	`+ // no Hibernate Validator 5.2+ or similar provider`
`271`	`271`	`}`
`272`	`272`	`}`
`273`	`273`
`@@ -417,8 +417,8 @@ public <T> T unwrap(@Nullable Class<T> type) {`
`417`	`417`	`try {`
`418`	`418`	`return super.unwrap(type);`
`419`	`419`	`}`
`420`		`- catch (ValidationException ex) {`
`421`		`- // Ignore - we'll try ValidatorFactory unwrapping next`
	`420`	`+ catch (ValidationException ignored) {`
	`421`	`+ // we'll try ValidatorFactory unwrapping next`
`422`	`422`	`}`
`423`	`423`	`}`
`424`	`424`	`if (this.validatorFactory != null) {`
Original file line number	Diff line number	Diff line change
`@@ -233,8 +233,7 @@ void serializableTargetAndAdvice() throws Throwable {`
`233`	`233`	`try {`
`234`	`234`	`p2.echo(new IOException());`
`235`	`235`	`}`
`236`		`- catch (IOException ex) {`
`237`		`-`
	`236`	`+ catch (IOException ignored) {`
`238`	`237`	`}`
`239`	`238`	`assertThat(cta.getCalls()).isEqualTo(2);`
`240`	`239`	`}`
Original file line number	Diff line number	Diff line change
`@@ -84,8 +84,7 @@ protected Class<?> findClass(String name) throws ClassNotFoundException {`
`84`	`84`	`try (stream) {`
`85`	`85`	`return stream.readAllBytes();`
`86`	`86`	`}`
`87`		`- catch (IOException ex) {`
`88`		`- // ignore`
	`87`	`+ catch (IOException ignored) {`
`89`	`88`	`}`
`90`	`89`	`}`
`91`	`90`	`return null;`
Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,8 @@ public BridgeMethodResolver(Map declToBridge, ClassLoader classLoader) {`
`73`	`73`	`} finally {`
`74`	`74`	`is.close();`
`75`	`75`	`}`
`76`		`- } catch (IOException ignored) {}`
	`76`	`+ } catch (IOException ignored) {`
	`77`	`+ }`
`77`	`78`	`}`
`78`	`79`	`return resolved;`
`79`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -62,8 +62,8 @@ public static MethodProxy create(Class c1, Class c2, String desc, String name1,`
`62`	`62`	`try {`
`63`	`63`	`proxy.init();`
`64`	`64`	`}`
`65`		`- catch (CodeGenerationException ex) {`
`66`		`- // Ignore - to be retried when actually needed later on (possibly not at all)`
	`65`	`+ catch (CodeGenerationException ignored) {`
	`66`	`+ // to be retried when actually needed later on (possibly not at all)`
`67`	`67`	`}`
`68`	`68`	`}`
`69`	`69`	`// SPRING PATCH END`
Original file line number	Diff line number	Diff line change
`@@ -438,8 +438,7 @@ private static class KotlinDelegate {`
`438`	`438`	`return constructor;`
`439`	`439`	`}`
`440`	`440`	`}`
`441`		`- catch (UnsupportedOperationException ex) {`
`442`		`- // ignore`
	`441`	`+ catch (UnsupportedOperationException ignored) {`
`443`	`442`	`}`
`444`	`443`	`return null;`
`445`	`444`	`}`