Polishing

Author: jhoeller

spring-beans/src/main/java/org/springframework/beans/factory/config/YamlProcessor.java

@@ -393,17 +393,14 @@ public enum ResolutionMethod {
 	 */
 	protected static class StrictMapAppenderConstructor extends Constructor {
 
-		public 	StrictMapAppenderConstructor() {
-			super();
-		}
-
 		@Override
 		protected Map<Object, Object> constructMapping(MappingNode node) {
 			try {
 				return super.constructMapping(node);
-			} catch (IllegalStateException e) {
+			}
+			catch (IllegalStateException ex) {
 				throw new ParserException("while parsing MappingNode",
-						node.getStartMark(), e.getMessage(), node.getEndMark());
+						node.getStartMark(), ex.getMessage(), node.getEndMark());
 			}
 		}
 
@@ -414,7 +411,7 @@ protected Map<Object, Object> createDefaultMap() {
 				@Override
 				public Object put(Object key, Object value) {
 					if (delegate.containsKey(key)) {
-						throw new IllegalStateException("duplicate key: " + key);
+						throw new IllegalStateException("Duplicate key: " + key);
 					}
 					return delegate.put(key, value);
 				}

spring-expression/src/main/java/org/springframework/expression/spel/support/ReflectiveMethodExecutor.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.
@@ -80,13 +80,14 @@ private Class<?> discoverPublicClass(Method method, Class<?> clazz) {
 			try {
 				clazz.getDeclaredMethod(method.getName(), method.getParameterTypes());
 				return clazz;
-			} catch (NoSuchMethodException nsme) {
-
+			}
+			catch (NoSuchMethodException ex) {
+				// Continue below...
 			}
 		}
-		Class<?>[] intfaces = clazz.getInterfaces();
-		for (Class<?> intface: intfaces) {
-			discoverPublicClass(method, intface);
+		Class<?>[] ifcs = clazz.getInterfaces();
+		for (Class<?> ifc: ifcs) {
+			discoverPublicClass(method, ifc);
 		}
 		if (clazz.getSuperclass() != null) {
 			return discoverPublicClass(method, clazz.getSuperclass());

spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/StompDecoder.java

@@ -58,9 +58,8 @@ public class StompDecoder {
 
 
 	/**
-	 * Configure a
-	 * {@link org.springframework.messaging.support.MessageHeaderInitializer MessageHeaderInitializer}
-	 * to apply to the headers of {@link Message}s from decoded STOMP frames.
+	 * Configure a {@link MessageHeaderInitializer} to apply to the headers of
+	 * {@link Message}s from decoded STOMP frames.
 	 */
 	public void setHeaderInitializer(MessageHeaderInitializer headerInitializer) {
 		this.headerInitializer = headerInitializer;
@@ -216,7 +215,7 @@ private void readHeaders(ByteBuffer buffer, StompHeaderAccessor headerAccessor)
 				String header = new String(headerStream.toByteArray(), UTF8_CHARSET);
 				int colonIndex = header.indexOf(':');
 				if (colonIndex <= 0) {
-					if(buffer.remaining() > 0) {
+					if (buffer.remaining() > 0) {
 						throw new StompConversionException("Illegal header: '" + header +
 								"'. A header must be of the form <name>:[<value>].");
 					}

spring-test/src/main/java/org/springframework/mock/web/MockHttpServletResponse.java

@@ -33,7 +33,6 @@
 import java.util.Locale;
 import java.util.Map;
 import java.util.TimeZone;
-
 import javax.servlet.ServletOutputStream;
 import javax.servlet.http.Cookie;
 import javax.servlet.http.HttpServletResponse;
@@ -499,7 +498,8 @@ public long getDateHeader(String name) {
 		dateFormat.setTimeZone(GMT);
 		try {
 			return dateFormat.parse(getHeader(name)).getTime();
-		} catch (ParseException exception) {
+		}
+		catch (ParseException ex) {
 			throw new IllegalArgumentException(
 					"Value for header '" + name + "' is not a valid Date: " + getHeader(name));
 		}

spring-test/src/test/java/org/springframework/test/web/servlet/samples/standalone/ResponseBodyTests.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.
@@ -38,22 +38,21 @@ public class ResponseBodyTests {
 
 	@Test
 	public void json() throws Exception {
-
 		standaloneSetup(new PersonController()).build()
-			.perform(get("/person/Lee").accept(MediaType.APPLICATION_JSON))
+				.perform(get("/person/Lee").accept(MediaType.APPLICATION_JSON))
 				.andExpect(status().isOk())
 				.andExpect(content().contentType("application/json;charset=UTF-8"))
 				.andExpect(jsonPath("$.name").value("Lee"));
 	}
 
+
 	@Controller
 	private class PersonController {
 
 		@RequestMapping(value="/person/{name}")
 		@ResponseBody
 		public Person get(@PathVariable String name) {
-			Person person = new Person(name);
-			return person;
+			return new Person(name);
 		}
 	}
 

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

@@ -422,10 +422,10 @@ public BodyBuilder contentType(MediaType contentType) {
 		@Override
 		public BodyBuilder eTag(String eTag) {
 			if (eTag != null) {
-				if(!eTag.startsWith("\"") && !eTag.startsWith("W/\"")) {
+				if (!eTag.startsWith("\"") && !eTag.startsWith("W/\"")) {
 					eTag = "\"" + eTag;
 				}
-				if(!eTag.endsWith("\"")) {
+				if (!eTag.endsWith("\"")) {
 					eTag = eTag + "\"";
 				}
 			}

spring-web/src/main/java/org/springframework/web/accept/ContentNegotiationManagerFactoryBean.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.
@@ -234,7 +234,7 @@ public void afterPropertiesSet() {
 			strategies.add(new HeaderContentNegotiationStrategy());
 		}
 
-		if(this.defaultNegotiationStrategy != null) {
+		if (this.defaultNegotiationStrategy != null) {
 			strategies.add(defaultNegotiationStrategy);
 		}
 

spring-web/src/main/java/org/springframework/web/client/HttpMessageConverterExtractor.java

@@ -80,7 +80,7 @@ public HttpMessageConverterExtractor(Type responseType, List<HttpMessageConverte
 	public T extractData(ClientHttpResponse response) throws IOException {
 
 		MessageBodyClientHttpResponseWrapper responseWrapper = new MessageBodyClientHttpResponseWrapper(response);
-		if(!responseWrapper.hasMessageBody() || responseWrapper.hasEmptyMessageBody()) {
+		if (!responseWrapper.hasMessageBody() || responseWrapper.hasEmptyMessageBody()) {
 			return null;
 		}
 		MediaType contentType = getContentType(responseWrapper);

spring-web/src/main/java/org/springframework/web/client/MessageBodyClientHttpResponseWrapper.java

@@ -61,7 +61,7 @@ public boolean hasMessageBody() throws IOException {
 				responseStatus == HttpStatus.NOT_MODIFIED) {
 			return false;
 		}
-		else if(this.getHeaders().getContentLength() == 0) {
+		else if (this.getHeaders().getContentLength() == 0) {
 			return false;
 		}
 		return true;

spring-web/src/main/java/org/springframework/web/util/UrlPathHelper.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.
@@ -180,7 +180,7 @@ public String getPathWithinServletMapping(HttpServletRequest request) {
 		String path;
 
 		// if the app container sanitized the servletPath, check against the sanitized version
-		if(servletPath.indexOf(sanitizedPathWithinApp) != -1) {
+		if (servletPath.indexOf(sanitizedPathWithinApp) != -1) {
 			path = getRemainingPath(sanitizedPathWithinApp, servletPath, false);
 		}
 		else {