Polishing

Author: jhoeller

spring-web/src/main/java/org/springframework/http/MediaType.java

@@ -284,16 +284,15 @@ public MediaType(String type, String subtype) {
 	 * Create a new {@code MediaType} for the given type, subtype, and character set.
 	 * @param type the primary type
 	 * @param subtype the subtype
-	 * @param charSet the character set
+	 * @param charset the character set
 	 * @throws IllegalArgumentException if any of the parameters contain illegal characters
 	 */
-	public MediaType(String type, String subtype, Charset charSet) {
-		this(type, subtype, Collections.singletonMap(PARAM_CHARSET, charSet.name()));
+	public MediaType(String type, String subtype, Charset charset) {
+		this(type, subtype, Collections.singletonMap(PARAM_CHARSET, charset.name()));
 	}
 
 	/**
 	 * Create a new {@code MediaType} for the given type, subtype, and quality value.
-	 *
 	 * @param type the primary type
 	 * @param subtype the subtype
 	 * @param qualityValue the quality value
@@ -774,18 +773,18 @@ public static String toString(Collection<MediaType> mediaTypes) {
 	 * Sorts the given list of {@code MediaType} objects by specificity.
 	 * <p>Given two media types:
 	 * <ol>
-	 *   <li>if either media type has a {@linkplain #isWildcardType() wildcard type}, then the media type without the
-	 *   wildcard is ordered before the other.</li>
-	 *   <li>if the two media types have different {@linkplain #getType() types}, then they are considered equal and
-	 *   remain their current order.</li>
-	 *   <li>if either media type has a {@linkplain #isWildcardSubtype() wildcard subtype}, then the media type without
-	 *   the wildcard is sorted before the other.</li>
-	 *   <li>if the two media types have different {@linkplain #getSubtype() subtypes}, then they are considered equal
-	 *   and remain their current order.</li>
-	 *   <li>if the two media types have different {@linkplain #getQualityValue() quality value}, then the media type
-	 *   with the highest quality value is ordered before the other.</li>
-	 *   <li>if the two media types have a different amount of {@linkplain #getParameter(String) parameters}, then the
-	 *   media type with the most parameters is ordered before the other.</li>
+	 * <li>if either media type has a {@linkplain #isWildcardType() wildcard type}, then the media type without the
+	 * wildcard is ordered before the other.</li>
+	 * <li>if the two media types have different {@linkplain #getType() types}, then they are considered equal and
+	 * remain their current order.</li>
+	 * <li>if either media type has a {@linkplain #isWildcardSubtype() wildcard subtype}, then the media type without
+	 * the wildcard is sorted before the other.</li>
+	 * <li>if the two media types have different {@linkplain #getSubtype() subtypes}, then they are considered equal
+	 * and remain their current order.</li>
+	 * <li>if the two media types have different {@linkplain #getQualityValue() quality value}, then the media type
+	 * with the highest quality value is ordered before the other.</li>
+	 * <li>if the two media types have a different amount of {@linkplain #getParameter(String) parameters}, then the
+	 * media type with the most parameters is ordered before the other.</li>
 	 * </ol>
 	 * <p>For example:
 	 * <blockquote>audio/basic &lt; audio/* &lt; *&#047;*</blockquote>
@@ -807,18 +806,18 @@ public static void sortBySpecificity(List<MediaType> mediaTypes) {
 	 * Sorts the given list of {@code MediaType} objects by quality value.
 	 * <p>Given two media types:
 	 * <ol>
-	 *   <li>if the two media types have different {@linkplain #getQualityValue() quality value}, then the media type
-	 *   with the highest quality value is ordered before the other.</li>
-	 *   <li>if either media type has a {@linkplain #isWildcardType() wildcard type}, then the media type without the
-	 *   wildcard is ordered before the other.</li>
-	 *   <li>if the two media types have different {@linkplain #getType() types}, then they are considered equal and
-	 *   remain their current order.</li>
-	 *   <li>if either media type has a {@linkplain #isWildcardSubtype() wildcard subtype}, then the media type without
-	 *   the wildcard is sorted before the other.</li>
-	 *   <li>if the two media types have different {@linkplain #getSubtype() subtypes}, then they are considered equal
-	 *   and remain their current order.</li>
-	 *   <li>if the two media types have a different amount of {@linkplain #getParameter(String) parameters}, then the
-	 *   media type with the most parameters is ordered before the other.</li>
+	 * <li>if the two media types have different {@linkplain #getQualityValue() quality value}, then the media type
+	 * with the highest quality value is ordered before the other.</li>
+	 * <li>if either media type has a {@linkplain #isWildcardType() wildcard type}, then the media type without the
+	 * wildcard is ordered before the other.</li>
+	 * <li>if the two media types have different {@linkplain #getType() types}, then they are considered equal and
+	 * remain their current order.</li>
+	 * <li>if either media type has a {@linkplain #isWildcardSubtype() wildcard subtype}, then the media type without
+	 * the wildcard is sorted before the other.</li>
+	 * <li>if the two media types have different {@linkplain #getSubtype() subtypes}, then they are considered equal
+	 * and remain their current order.</li>
+	 * <li>if the two media types have a different amount of {@linkplain #getParameter(String) parameters}, then the
+	 * media type with the most parameters is ordered before the other.</li>
 	 * </ol>
 	 * @param mediaTypes the list of media types to be sorted
 	 * @see #getQualityValue()

spring-web/src/main/java/org/springframework/web/method/support/HandlerMethodArgumentResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2013 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.
@@ -33,7 +33,6 @@ public interface HandlerMethodArgumentResolver {
 	/**
 	 * Whether the given {@linkplain MethodParameter method parameter} is
 	 * supported by this resolver.
-	 *
 	 * @param parameter the method parameter to check
 	 * @return {@code true} if this resolver supports the supplied parameter;
 	 * {@code false} otherwise
@@ -46,7 +45,6 @@ public interface HandlerMethodArgumentResolver {
 	 * request. A {@link WebDataBinderFactory} provides a way to create
 	 * a {@link WebDataBinder} instance when needed for data binding and
 	 * type conversion purposes.
-	 *
 	 * @param parameter the method parameter to resolve. This parameter must
 	 * have previously been passed to
 	 * {@link #supportsParameter(org.springframework.core.MethodParameter)}
@@ -57,9 +55,7 @@ public interface HandlerMethodArgumentResolver {
 	 * @return the resolved argument value, or {@code null}.
 	 * @throws Exception in case of errors with the preparation of argument values
 	 */
-	Object resolveArgument(MethodParameter parameter,
-						   ModelAndViewContainer mavContainer,
-						   NativeWebRequest webRequest,
-						   WebDataBinderFactory binderFactory) throws Exception;
+	Object resolveArgument(MethodParameter parameter, ModelAndViewContainer mavContainer,
+			NativeWebRequest webRequest, WebDataBinderFactory binderFactory) throws Exception;
 
 }

spring-web/src/main/java/org/springframework/web/method/support/HandlerMethodReturnValueHandler.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2012 the original author or authors.
+ * Copyright 2002-2013 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.
@@ -31,7 +31,6 @@ public interface HandlerMethodReturnValueHandler {
 	/**
 	 * Whether the given {@linkplain MethodParameter method return type} is
 	 * supported by this handler.
-	 *
 	 * @param returnType the method return type to check
 	 * @return {@code true} if this handler supports the supplied return type;
 	 * {@code false} otherwise
@@ -43,7 +42,6 @@ public interface HandlerMethodReturnValueHandler {
 	 * setting a view or setting the
 	 * {@link ModelAndViewContainer#setRequestHandled} flag to {@code true}
 	 * to indicate the response has been handled directly.
-	 *
 	 * @param returnValue the value returned from the handler method
 	 * @param returnType the type of the return value. This type must have
 	 * previously been passed to
@@ -53,9 +51,7 @@ public interface HandlerMethodReturnValueHandler {
 	 * @param webRequest the current request
 	 * @throws Exception if the return value handling results in an error
 	 */
-	void handleReturnValue(Object returnValue,
-						   MethodParameter returnType,
-						   ModelAndViewContainer mavContainer,
-						   NativeWebRequest webRequest) throws Exception;
+	void handleReturnValue(Object returnValue, MethodParameter returnType,
+			ModelAndViewContainer mavContainer, NativeWebRequest webRequest) throws Exception;
 
 }