diff --git a/spring-core/src/main/java/org/springframework/core/MethodParameter.java b/spring-core/src/main/java/org/springframework/core/MethodParameter.java index 5c7e9bc7f5..a9ace68956 100644 --- a/spring-core/src/main/java/org/springframework/core/MethodParameter.java +++ b/spring-core/src/main/java/org/springframework/core/MethodParameter.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2012 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. @@ -310,14 +310,14 @@ public class MethodParameter { * Return true if the parameter has at least one annotation, false if it has none. */ public boolean hasParameterAnnotations() { - return getParameterAnnotations().length != 0; + return (getParameterAnnotations().length != 0); } /** * Return true if the parameter has the given annotation type, and false if it doesn't. */ public boolean hasParameterAnnotation(Class annotationType) { - return getParameterAnnotation(annotationType) != null; + return (getParameterAnnotation(annotationType) != null); } /** @@ -416,6 +416,38 @@ public class MethodParameter { return this.typeIndexesPerLevel; } + @Override + public boolean equals(Object obj) { + if (this == obj) { + return true; + } + if (obj != null && obj instanceof MethodParameter) { + MethodParameter other = (MethodParameter) obj; + + if (this.parameterIndex != other.parameterIndex) { + return false; + } + else if (this.getMember().equals(other.getMember())) { + return true; + } + else { + return false; + } + } + return false; + } + + @Override + public int hashCode() { + int result = this.hash; + if (result == 0) { + result = getMember().hashCode(); + result = 31 * result + this.parameterIndex; + this.hash = result; + } + return result; + } + /** * Create a new MethodParameter for the given method or constructor. @@ -438,37 +470,4 @@ public class MethodParameter { } } - @Override - public boolean equals(Object obj) { - if (this == obj) { - return true; - } - if (obj != null && obj instanceof MethodParameter) { - MethodParameter other = (MethodParameter) obj; - - if (this.parameterIndex != other.parameterIndex) { - return false; - } - else if (this.getMember().equals(other.getMember())) { - return true; - } - else { - return false; - } - } - return false; - } - - - @Override - public int hashCode() { - int result = this.hash; - if (result == 0) { - result = getMember().hashCode(); - result = 31 * result + this.parameterIndex; - this.hash = result; - } - return result; - } - } diff --git a/spring-core/src/test/java/org/springframework/core/MethodParameterTests.java b/spring-core/src/test/java/org/springframework/core/MethodParameterTests.java index 2f1ce99bfd..c3c3e7d139 100644 --- a/spring-core/src/test/java/org/springframework/core/MethodParameterTests.java +++ b/spring-core/src/test/java/org/springframework/core/MethodParameterTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2011 the original author or authors. + * Copyright 2002-2012 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. @@ -34,6 +34,7 @@ public class MethodParameterTests { private MethodParameter intReturnType; + @Before public void setUp() throws NoSuchMethodException { Method method = getClass().getMethod("method", String.class, Long.TYPE); @@ -42,8 +43,6 @@ public class MethodParameterTests { intReturnType = new MethodParameter(method, -1); } - - @Test public void testEquals() throws NoSuchMethodException { assertEquals(stringParameter, stringParameter); diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/multiaction/MultiActionController.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/multiaction/MultiActionController.java index 178f0fcf89..f88c921c91 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/multiaction/MultiActionController.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/multiaction/MultiActionController.java @@ -1,5 +1,5 @@ /* - * Copyright 2002-2008 the original author or authors. + * Copyright 2002-2012 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. @@ -513,7 +513,7 @@ public class MultiActionController extends AbstractController implements LastMod * @throws Exception if the command object could not be instantiated * @see org.springframework.beans.BeanUtils#instantiateClass(Class) */ - protected Object newCommandObject(Class clazz) throws Exception { + protected Object newCommandObject(Class clazz) throws Exception { if (logger.isDebugEnabled()) { logger.debug("Creating new command of class [" + clazz.getName() + "]"); }