Use Arrays.toString instead of Arrays.asList when generating Strings

This commit is contained in:
Sam Brannen 2022-05-17 15:34:09 +02:00
parent a1c3efbb5f
commit 59c7bb1f86
7 changed files with 14 additions and 14 deletions

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -378,7 +378,7 @@ public class AspectJExpressionPointcut extends AbstractExpressionPointcut
} }
catch (Throwable ex) { catch (Throwable ex) {
if (logger.isDebugEnabled()) { if (logger.isDebugEnabled()) {
logger.debug("Failed to evaluate join point for arguments " + Arrays.asList(args) + logger.debug("Failed to evaluate join point for arguments " + Arrays.toString(args) +
" - falling back to non-match", ex); " - falling back to non-match", ex);
} }
return false; return false;

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -424,7 +424,7 @@ public class ProxyFactoryBean extends ProxyCreatorSupport
if (!this.advisorChainInitialized && !ObjectUtils.isEmpty(this.interceptorNames)) { if (!this.advisorChainInitialized && !ObjectUtils.isEmpty(this.interceptorNames)) {
if (this.beanFactory == null) { if (this.beanFactory == null) {
throw new IllegalStateException("No BeanFactory available anymore (probably due to serialization) " + throw new IllegalStateException("No BeanFactory available anymore (probably due to serialization) " +
"- cannot resolve interceptor names " + Arrays.asList(this.interceptorNames)); "- cannot resolve interceptor names " + Arrays.toString(this.interceptorNames));
} }
// Globals can't be last unless we specified a targetSource using the property... // Globals can't be last unless we specified a targetSource using the property...

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2018 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -143,7 +143,7 @@ public class NotificationListenerRegistrar extends NotificationListenerHolder
this.actualObjectNames = getResolvedObjectNames(); this.actualObjectNames = getResolvedObjectNames();
if (this.actualObjectNames != null) { if (this.actualObjectNames != null) {
if (logger.isDebugEnabled()) { if (logger.isDebugEnabled()) {
logger.debug("Registering NotificationListener for MBeans " + Arrays.asList(this.actualObjectNames)); logger.debug("Registering NotificationListener for MBeans " + Arrays.toString(this.actualObjectNames));
} }
for (ObjectName actualObjectName : this.actualObjectNames) { for (ObjectName actualObjectName : this.actualObjectNames) {
this.server.addNotificationListener( this.server.addNotificationListener(

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -299,7 +299,7 @@ public abstract class AbstractEnvironment implements ConfigurableEnvironment {
public void setActiveProfiles(String... profiles) { public void setActiveProfiles(String... profiles) {
Assert.notNull(profiles, "Profile array must not be null"); Assert.notNull(profiles, "Profile array must not be null");
if (logger.isDebugEnabled()) { if (logger.isDebugEnabled()) {
logger.debug("Activating profiles " + Arrays.asList(profiles)); logger.debug("Activating profiles " + Arrays.toString(profiles));
} }
synchronized (this.activeProfiles) { synchronized (this.activeProfiles) {
this.activeProfiles.clear(); this.activeProfiles.clear();

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2021 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -451,7 +451,7 @@ public abstract class AbstractJdbcInsert {
if (getGeneratedKeyNames().length > 1) { if (getGeneratedKeyNames().length > 1) {
throw new InvalidDataAccessApiUsageException( throw new InvalidDataAccessApiUsageException(
"Current database only supports retrieving the key for a single column. There are " + "Current database only supports retrieving the key for a single column. There are " +
getGeneratedKeyNames().length + " columns specified: " + Arrays.asList(getGeneratedKeyNames())); getGeneratedKeyNames().length + " columns specified: " + Arrays.toString(getGeneratedKeyNames()));
} }
Assert.state(getTableName() != null, "No table name set"); Assert.state(getTableName() != null, "No table name set");

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2017 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -118,7 +118,7 @@ public class SpringConfigurator extends Configurator {
beanNamesByType.put(endpointClass, NO_VALUE); beanNamesByType.put(endpointClass, NO_VALUE);
if (names.length > 1) { if (names.length > 1) {
throw new IllegalStateException("Found multiple @ServerEndpoint's of type [" + throw new IllegalStateException("Found multiple @ServerEndpoint's of type [" +
endpointClass.getName() + "]: bean names " + Arrays.asList(names)); endpointClass.getName() + "]: bean names " + Arrays.toString(names));
} }
} }
} }

View File

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2019 the original author or authors. * Copyright 2002-2022 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -82,7 +82,7 @@ public abstract class AbstractHttpReceivingTransportHandler extends AbstractTran
return; return;
} }
if (logger.isTraceEnabled()) { if (logger.isTraceEnabled()) {
logger.trace("Received message(s): " + Arrays.asList(messages)); logger.trace("Received message(s): " + Arrays.toString(messages));
} }
response.setStatusCode(getResponseStatus()); response.setStatusCode(getResponseStatus());
response.getHeaders().setContentType(new MediaType("text", "plain", StandardCharsets.UTF_8)); response.getHeaders().setContentType(new MediaType("text", "plain", StandardCharsets.UTF_8));