From 753347ea98929922c4df84b87de8a3ee1195187a Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Wed, 16 Dec 2015 20:44:00 +0100 Subject: [PATCH] Consistent static final logger declarations --- .../aop/framework/CglibAopProxy.java | 2 +- .../jmx/access/ConnectorDelegate.java | 4 ++-- .../SubscriptionMethodReturnValueHandler.java | 2 +- .../socket/config/WebSocketMessageBrokerStats.java | 9 ++++----- .../web/socket/messaging/WebSocketStompClient.java | 2 +- .../sockjs/client/DefaultTransportRequest.java | 10 +++++----- .../socket/sockjs/client/WebSocketTransport.java | 14 +++++++------- 7 files changed, 21 insertions(+), 22 deletions(-) diff --git a/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java b/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java index ffe461fdb01..7e60723e031 100644 --- a/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java +++ b/spring-aop/src/main/java/org/springframework/aop/framework/CglibAopProxy.java @@ -94,7 +94,7 @@ class CglibAopProxy implements AopProxy, Serializable { /** Logger available to subclasses; static to optimize serialization */ - protected final static Log logger = LogFactory.getLog(CglibAopProxy.class); + protected static final Log logger = LogFactory.getLog(CglibAopProxy.class); /** Keeps track of the Classes that we have validated for final methods */ private static final Map, Boolean> validatedClasses = new WeakHashMap, Boolean>(); diff --git a/spring-context/src/main/java/org/springframework/jmx/access/ConnectorDelegate.java b/spring-context/src/main/java/org/springframework/jmx/access/ConnectorDelegate.java index bd99b693935..8490d680202 100644 --- a/spring-context/src/main/java/org/springframework/jmx/access/ConnectorDelegate.java +++ b/spring-context/src/main/java/org/springframework/jmx/access/ConnectorDelegate.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2012 the original author or authors. + * Copyright 2002-2015 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. @@ -37,7 +37,7 @@ import org.springframework.jmx.support.JmxUtils; */ class ConnectorDelegate { - private final static Log logger = LogFactory.getLog(ConnectorDelegate.class); + private static final Log logger = LogFactory.getLog(ConnectorDelegate.class); private JMXConnector connector; diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SubscriptionMethodReturnValueHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SubscriptionMethodReturnValueHandler.java index 085446fab9a..c762a130add 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SubscriptionMethodReturnValueHandler.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SubscriptionMethodReturnValueHandler.java @@ -50,7 +50,7 @@ import org.springframework.util.Assert; */ public class SubscriptionMethodReturnValueHandler implements HandlerMethodReturnValueHandler { - private static Log logger = LogFactory.getLog(SubscriptionMethodReturnValueHandler.class); + private static final Log logger = LogFactory.getLog(SubscriptionMethodReturnValueHandler.class); private final MessageSendingOperations messagingTemplate; diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/config/WebSocketMessageBrokerStats.java b/spring-websocket/src/main/java/org/springframework/web/socket/config/WebSocketMessageBrokerStats.java index b3b446e6d7b..0f5bd934e8c 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/config/WebSocketMessageBrokerStats.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/config/WebSocketMessageBrokerStats.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2014 the original author or authors. + * Copyright 2002-2015 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. @@ -32,7 +32,6 @@ import org.springframework.web.socket.messaging.StompSubProtocolHandler; import org.springframework.web.socket.messaging.SubProtocolHandler; import org.springframework.web.socket.messaging.SubProtocolWebSocketHandler; - /** * A central class for aggregating information about internal state and counters * from key infrastructure components of the setup that comes with @@ -51,7 +50,7 @@ import org.springframework.web.socket.messaging.SubProtocolWebSocketHandler; */ public class WebSocketMessageBrokerStats { - private static Log logger = LogFactory.getLog(WebSocketMessageBrokerStats.class); + private static final Log logger = LogFactory.getLog(WebSocketMessageBrokerStats.class); private SubProtocolWebSocketHandler webSocketHandler; @@ -181,8 +180,8 @@ public class WebSocketMessageBrokerStats { } private String getExecutorStatsInfo(Executor executor) { - String s = executor.toString(); - return s.substring(s.indexOf("pool"), s.length() - 1); + String str = executor.toString(); + return str.substring(str.indexOf("pool"), str.length() - 1); } public String toString() { diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketStompClient.java b/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketStompClient.java index d62234f9af3..1e648d62c9b 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketStompClient.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/messaging/WebSocketStompClient.java @@ -70,7 +70,7 @@ import org.springframework.web.util.UriComponentsBuilder; */ public class WebSocketStompClient extends StompClientSupport implements SmartLifecycle { - private static Log logger = LogFactory.getLog(WebSocketStompClient.class); + private static final Log logger = LogFactory.getLog(WebSocketStompClient.class); private final WebSocketClient webSocketClient; diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/DefaultTransportRequest.java b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/DefaultTransportRequest.java index 06563ad6339..99949da1c05 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/DefaultTransportRequest.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/DefaultTransportRequest.java @@ -45,7 +45,7 @@ import org.springframework.web.socket.sockjs.transport.TransportType; */ class DefaultTransportRequest implements TransportRequest { - private static Log logger = LogFactory.getLog(DefaultTransportRequest.class); + private static final Log logger = LogFactory.getLog(DefaultTransportRequest.class); private final SockJsUrlInfo sockJsUrlInfo; @@ -75,10 +75,10 @@ class DefaultTransportRequest implements TransportRequest { HttpHeaders handshakeHeaders, HttpHeaders httpRequestHeaders, Transport transport, TransportType serverTransportType, SockJsMessageCodec codec) { - Assert.notNull(sockJsUrlInfo, "'sockJsUrlInfo' is required"); - Assert.notNull(transport, "'transport' is required"); - Assert.notNull(serverTransportType, "'transportType' is required"); - Assert.notNull(codec, "'codec' is required"); + Assert.notNull(sockJsUrlInfo, "SockJsUrlInfo is required"); + Assert.notNull(transport, "Transport is required"); + Assert.notNull(serverTransportType, "TransportType is required"); + Assert.notNull(codec, "SockJsMessageCodec is required"); this.sockJsUrlInfo = sockJsUrlInfo; this.handshakeHeaders = (handshakeHeaders != null ? handshakeHeaders : new HttpHeaders()); this.httpRequestHeaders = (httpRequestHeaders != null ? httpRequestHeaders : new HttpHeaders()); diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/WebSocketTransport.java b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/WebSocketTransport.java index 35f01befd17..1a7dc43f3c1 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/WebSocketTransport.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/WebSocketTransport.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2014 the original author or authors. + * Copyright 2002-2015 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,7 +17,7 @@ package org.springframework.web.socket.sockjs.client; import java.net.URI; -import java.util.Arrays; +import java.util.Collections; import java.util.List; import java.util.concurrent.atomic.AtomicInteger; @@ -47,7 +47,7 @@ import org.springframework.web.socket.sockjs.transport.TransportType; */ public class WebSocketTransport implements Transport, Lifecycle { - private static Log logger = LogFactory.getLog(WebSocketTransport.class); + private static final Log logger = LogFactory.getLog(WebSocketTransport.class); private final WebSocketClient webSocketClient; @@ -55,7 +55,7 @@ public class WebSocketTransport implements Transport, Lifecycle { public WebSocketTransport(WebSocketClient webSocketClient) { - Assert.notNull(webSocketClient, "'webSocketClient' is required"); + Assert.notNull(webSocketClient, "WebSocketClient is required"); this.webSocketClient = webSocketClient; } @@ -69,7 +69,7 @@ public class WebSocketTransport implements Transport, Lifecycle { @Override public List getTransportTypes() { - return Arrays.asList(TransportType.WEBSOCKET); + return Collections.singletonList(TransportType.WEBSOCKET); } @Override @@ -146,7 +146,7 @@ public class WebSocketTransport implements Transport, Lifecycle { private final AtomicInteger connectCount = new AtomicInteger(0); - private ClientSockJsWebSocketHandler(WebSocketClientSockJsSession session) { + public ClientSockJsWebSocketHandler(WebSocketClientSockJsSession session) { Assert.notNull(session); this.sockJsSession = session; } @@ -173,4 +173,4 @@ public class WebSocketTransport implements Transport, Lifecycle { } } -} \ No newline at end of file +}