Author: markt
Date: Wed Jun 26 12:06:45 2013
New Revision: 1496897

URL: http://svn.apache.org/r1496897
Log:
Extract decoder use flag to a field (will use it later)
Modified:
    tomcat/trunk/java/org/apache/tomcat/websocket/pojo/PojoMethodMapping.java

Modified: 
tomcat/trunk/java/org/apache/tomcat/websocket/pojo/PojoMethodMapping.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/pojo/PojoMethodMapping.java?rev=1496897&r1=1496896&r2=1496897&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/pojo/PojoMethodMapping.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/pojo/PojoMethodMapping.java 
Wed Jun 26 12:06:45 2013
@@ -267,6 +267,7 @@ public class PojoMethodMapping {
         private int indexPrimitive = -1;
         private Map<Integer,PojoPathParam> indexPathParams = new HashMap<>();
         private int indexPayload = -1;
+        private boolean useDecoder = false;
 
 
         public MessageMethod(Method m, List<DecoderEntry> decoderEntries) {
@@ -274,7 +275,6 @@ public class PojoMethodMapping {
 
             Class<?>[] types = m.getParameterTypes();
             Annotation[][] paramsAnnotations = m.getParameterAnnotations();
-            boolean decoderParameterFound = false;
 
             for (int i = 0; i < types.length; i++) {
                 boolean paramFound = false;
@@ -378,7 +378,7 @@ public class PojoMethodMapping {
                                     if (indexByteBuffer == -1) {
                                         indexByteBuffer = i;
                                         foundBinaryDecoderMatch = true;
-                                        decoderParameterFound = true;
+                                        useDecoder = true;
                                     } else {
                                         throw new 
IllegalArgumentException(sm.getString(
                                                 
"pojoMethodMapping.duplicateMessageParam",
@@ -390,7 +390,7 @@ public class PojoMethodMapping {
                                     if (indexString == -1) {
                                         indexString = i;
                                         foundTextDecoderMatch = true;
-                                        decoderParameterFound = true;
+                                        useDecoder = true;
                                     } else {
                                         throw new 
IllegalArgumentException(sm.getString(
                                                 
"pojoMethodMapping.duplicateMessageParam",
@@ -487,7 +487,7 @@ public class PojoMethodMapping {
                         "pojoMethodMapping.partialInputStream",
                         m.getName(), m.getDeclaringClass().getName()));
             }
-            if (decoderParameterFound && indexBoolean != -1) {
+            if (useDecoder && indexBoolean != -1) {
                 throw new IllegalArgumentException(sm.getString(
                         "pojoMethodMapping.partialObject",
                         m.getName(), m.getDeclaringClass().getName()));



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to