You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2012/12/04 16:51:10 UTC
svn commit: r1417003 -
/tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java
Author: kkolinko
Date: Tue Dec 4 15:51:09 2012
New Revision: 1417003
URL: http://svn.apache.org/viewvc?rev=1417003&view=rev
Log:
Review of r1416258
'&&' has higher priority than '||', so the brackets are needed.
Modified:
tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java
Modified: tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java?rev=1417003&r1=1417002&r2=1417003&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/PojoMethodMapping.java Tue Dec 4 15:51:09 2012
@@ -184,7 +184,7 @@ public class PojoMethodMapping {
} else if (type.equals(byte.class) || type.equals(Byte.class)) {
return Byte.valueOf(value);
} else if (value.length() == 1 &&
- type.equals(char.class) || type.equals(Character.class)) {
+ (type.equals(char.class) || type.equals(Character.class))) {
return Character.valueOf(value.charAt(0));
} else if (type.equals(double.class) || type.equals(Double.class)) {
return Double.valueOf(value);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org