From 3bd215404699d362c58af75e18d3831dd0ef3269 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Tue, 19 Sep 2017 00:02:40 +0200 Subject: [PATCH] Polishing --- .../messaging/core/AbstractMessageSendingTemplate.java | 2 +- .../web/server/session/InMemoryWebSessionStore.java | 9 ++++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/spring-messaging/src/main/java/org/springframework/messaging/core/AbstractMessageSendingTemplate.java b/spring-messaging/src/main/java/org/springframework/messaging/core/AbstractMessageSendingTemplate.java index 79c46e94d3..082e6b7d8c 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/core/AbstractMessageSendingTemplate.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/core/AbstractMessageSendingTemplate.java @@ -81,7 +81,7 @@ public abstract class AbstractMessageSendingTemplate implements MessageSendin * @param messageConverter the message converter to use */ public void setMessageConverter(MessageConverter messageConverter) { - Assert.notNull(messageConverter, "'messageConverter' must not be null"); + Assert.notNull(messageConverter, "MessageConverter must not be null"); this.converter = messageConverter; } diff --git a/spring-web/src/main/java/org/springframework/web/server/session/InMemoryWebSessionStore.java b/spring-web/src/main/java/org/springframework/web/server/session/InMemoryWebSessionStore.java index 3b833490f9..f3a5758596 100644 --- a/spring-web/src/main/java/org/springframework/web/server/session/InMemoryWebSessionStore.java +++ b/spring-web/src/main/java/org/springframework/web/server/session/InMemoryWebSessionStore.java @@ -13,6 +13,7 @@ * See the License for the specific language governing permissions and * limitations under the License. */ + package org.springframework.web.server.session; import java.time.Clock; @@ -57,7 +58,7 @@ public class InMemoryWebSessionStore implements WebSessionStore { * @param clock the clock to use */ public void setClock(Clock clock) { - Assert.notNull(clock, "'clock' is required."); + Assert.notNull(clock, "Clock is required"); this.clock = clock; } @@ -93,7 +94,8 @@ public class InMemoryWebSessionStore implements WebSessionStore { }); } - /* Private methods for InMemoryWebSession */ + + // Private methods for InMemoryWebSession private Mono changeSessionId(String oldId, WebSession session) { this.sessions.remove(oldId); @@ -121,7 +123,6 @@ public class InMemoryWebSessionStore implements WebSessionStore { private volatile boolean started; - InMemoryWebSession() { this.id = new AtomicReference<>(String.valueOf(idGenerator.generateId())); this.attributes = new ConcurrentHashMap<>(); @@ -139,7 +140,6 @@ public class InMemoryWebSessionStore implements WebSessionStore { this.started = existingSession.isStarted(); // Use method (explicit or implicit start) } - @Override public String getId() { return this.id.get(); @@ -170,7 +170,6 @@ public class InMemoryWebSessionStore implements WebSessionStore { return this.maxIdleTime; } - @Override public void start() { this.started = true;