Browse Source

Consistent final logger fields

(cherry picked from commit cfa0f6c)
pull/1104/merge
Juergen Hoeller 9 years ago
parent
commit
f2ac416f66
  1. 4
      spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java
  2. 4
      spring-websocket/src/main/java/org/springframework/web/socket/server/support/OriginHandshakeInterceptor.java
  3. 4
      spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/AbstractXhrTransport.java

4
spring-aop/src/main/java/org/springframework/aop/target/dynamic/AbstractRefreshableTargetSource.java

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2012 the original author or authors. * Copyright 2002-2016 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.
@ -39,7 +39,7 @@ import org.springframework.aop.TargetSource;
public abstract class AbstractRefreshableTargetSource implements TargetSource, Refreshable { public abstract class AbstractRefreshableTargetSource implements TargetSource, Refreshable {
/** Logger available to subclasses */ /** Logger available to subclasses */
protected Log logger = LogFactory.getLog(getClass()); protected final Log logger = LogFactory.getLog(getClass());
protected Object targetObject; protected Object targetObject;

4
spring-websocket/src/main/java/org/springframework/web/socket/server/support/OriginHandshakeInterceptor.java

@ -1,5 +1,5 @@
/* /*
* Copyright 2002-2015 the original author or authors. * Copyright 2002-2016 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.
@ -42,7 +42,7 @@ import org.springframework.web.util.WebUtils;
*/ */
public class OriginHandshakeInterceptor implements HandshakeInterceptor { public class OriginHandshakeInterceptor implements HandshakeInterceptor {
protected Log logger = LogFactory.getLog(getClass()); protected final Log logger = LogFactory.getLog(getClass());
private final Set<String> allowedOrigins = new LinkedHashSet<String>(); private final Set<String> allowedOrigins = new LinkedHashSet<String>();

4
spring-websocket/src/main/java/org/springframework/web/socket/sockjs/client/AbstractXhrTransport.java

@ -55,7 +55,7 @@ public abstract class AbstractXhrTransport implements XhrTransport {
} }
protected Log logger = LogFactory.getLog(getClass()); protected final Log logger = LogFactory.getLog(getClass());
private boolean xhrStreamingDisabled; private boolean xhrStreamingDisabled;
@ -73,11 +73,9 @@ public abstract class AbstractXhrTransport implements XhrTransport {
* An {@code XhrTransport} can support both the "xhr_streaming" and "xhr" * An {@code XhrTransport} can support both the "xhr_streaming" and "xhr"
* SockJS server transports. From a client perspective there is no * SockJS server transports. From a client perspective there is no
* implementation difference. * implementation difference.
*
* <p>Typically an {@code XhrTransport} is used as "XHR streaming" first and * <p>Typically an {@code XhrTransport} is used as "XHR streaming" first and
* then, if that fails, as "XHR". In some cases however it may be helpful to * then, if that fails, as "XHR". In some cases however it may be helpful to
* suppress XHR streaming so that only XHR is attempted. * suppress XHR streaming so that only XHR is attempted.
*
* <p>By default this property is set to {@code false} which means both * <p>By default this property is set to {@code false} which means both
* "XHR streaming" and "XHR" apply. * "XHR streaming" and "XHR" apply.
*/ */

Loading…
Cancel
Save