From bc075c713f6524b8e6c15c11aea9e61985052a18 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Mon, 29 Dec 2014 20:15:54 +0100 Subject: [PATCH] Polishing --- .../beans/MutablePropertyValues.java | 1 + .../jmx/access/MBeanClientInterceptor.java | 1 + .../AbstractMethodMessageHandler.java | 3 +-- .../SimpAnnotationMethodMessageHandler.java | 21 ++++++++----------- 4 files changed, 12 insertions(+), 14 deletions(-) diff --git a/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java b/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java index d5653a3261..efb4ae9bbd 100644 --- a/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java +++ b/spring-beans/src/main/java/org/springframework/beans/MutablePropertyValues.java @@ -263,6 +263,7 @@ public class MutablePropertyValues implements PropertyValues, Serializable { * Get the raw property value, if any. * @param propertyName the name to search for * @return the raw property value, or {@code null} + * @since 4.0 * @see #getPropertyValue(String) * @see PropertyValue#getValue() */ diff --git a/spring-context/src/main/java/org/springframework/jmx/access/MBeanClientInterceptor.java b/spring-context/src/main/java/org/springframework/jmx/access/MBeanClientInterceptor.java index b07c160eb4..fb59a36291 100644 --- a/spring-context/src/main/java/org/springframework/jmx/access/MBeanClientInterceptor.java +++ b/spring-context/src/main/java/org/springframework/jmx/access/MBeanClientInterceptor.java @@ -604,6 +604,7 @@ public class MBeanClientInterceptor this.connector.close(); } + /** * Simple wrapper class around a method name and its signature. * Used as the key when caching methods. diff --git a/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java index 63705fe870..c7bef5bf62 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/handler/invocation/AbstractMethodMessageHandler.java @@ -93,8 +93,7 @@ public abstract class AbstractMethodMessageHandler * one of the configured prefixes are eligible for handling. When there is a * match the prefix is removed and only the remaining part of the destination * is used for method-mapping purposes. - * - *

By default no prefixes are configured in which case all messages are + *

By default, no prefixes are configured in which case all messages are * eligible for handling. */ public void setDestinationPrefixes(Collection prefixes) { diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SimpAnnotationMethodMessageHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SimpAnnotationMethodMessageHandler.java index bfcb51face..a1991072aa 100644 --- a/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SimpAnnotationMethodMessageHandler.java +++ b/spring-messaging/src/main/java/org/springframework/messaging/simp/annotation/support/SimpAnnotationMethodMessageHandler.java @@ -135,7 +135,6 @@ public class SimpAnnotationMethodMessageHandler extends AbstractMethodMessageHan *

Destination prefixes are expected to be slash-separated Strings and * therefore a slash is automatically appended where missing to ensure a * proper prefix-based match (i.e. matching complete segments). - * *

Note however that the remaining portion of a destination after the * prefix may use a different separator (e.g. commonly "." in messaging) * depending on the configured {@code PathMatcher}. @@ -160,10 +159,9 @@ public class SimpAnnotationMethodMessageHandler extends AbstractMethodMessageHan } /** - * Configure a {@link MessageConverter} to use to convert the payload of a message - * from serialize form with a specific MIME type to an Object matching the target - * method parameter. The converter is also used when sending message to the message - * broker. + * Configure a {@link MessageConverter} to use to convert the payload of a message from + * its serialized form with a specific MIME type to an Object matching the target method + * parameter. The converter is also used when sending a message to the message broker. * @see CompositeMessageConverter */ public void setMessageConverter(MessageConverter converter) { @@ -181,9 +179,9 @@ public class SimpAnnotationMethodMessageHandler extends AbstractMethodMessageHan } /** - * Configure a {@link ConversionService} to use when resolving method arguments, for - * example message header values. - *

By default an instance of {@link DefaultFormattingConversionService} is used. + * Configure a {@link ConversionService} to use when resolving method arguments, + * for example message header values. + *

By default, {@link DefaultFormattingConversionService} is used. */ public void setConversionService(ConversionService conversionService) { this.conversionService = conversionService; @@ -199,7 +197,7 @@ public class SimpAnnotationMethodMessageHandler extends AbstractMethodMessageHan /** * Set the PathMatcher implementation to use for matching destinations * against configured destination patterns. - *

By default AntPathMatcher is used + *

By default, {@link AntPathMatcher} is used. */ public void setPathMatcher(PathMatcher pathMatcher) { Assert.notNull(pathMatcher, "PathMatcher must not be null"); @@ -234,15 +232,14 @@ public class SimpAnnotationMethodMessageHandler extends AbstractMethodMessageHan * Configure a {@link MessageHeaderInitializer} to pass on to * {@link org.springframework.messaging.handler.invocation.HandlerMethodReturnValueHandler}s * that send messages from controller return values. - * - *

By default this property is not set. + *

By default, this property is not set. */ public void setHeaderInitializer(MessageHeaderInitializer headerInitializer) { this.headerInitializer = headerInitializer; } /** - * @return the configured header initializer. + * Return the configured header initializer. */ public MessageHeaderInitializer getHeaderInitializer() { return this.headerInitializer;