Fix missing break statements in Java ProtocolUtil
authorMark Slee <mcslee@apache.org>
Tue, 2 Oct 2007 04:43:31 +0000 (04:43 +0000)
committerMark Slee <mcslee@apache.org>
Tue, 2 Oct 2007 04:43:31 +0000 (04:43 +0000)
Summary: Oops. switch statement n00b error.

Reviewed By: jssarma

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665286 13f79535-47bb-0310-9956-ffa450edef68

lib/java/src/protocol/TProtocolUtil.java

index 935163c..ad333bc 100644 (file)
@@ -23,30 +23,37 @@ public class TProtocolUtil {
     case TType.BOOL:
       {
         prot.readBool();
+        break;
       }
     case TType.BYTE:
       {
         prot.readByte();
+        break;
       }
     case TType.I16:
       {
         prot.readI16();
+        break;
       }
     case TType.I32:
       {
         prot.readI32();
+        break;
       }
     case TType.I64:
       {
         prot.readI64();
+        break;
       }
     case TType.DOUBLE:
       {
         prot.readDouble();
+        break;
       }
     case TType.STRING:
       {
         prot.readString();
+        break;
       }
     case TType.STRUCT:
       {
@@ -60,6 +67,7 @@ public class TProtocolUtil {
           prot.readFieldEnd();
         }
         prot.readStructEnd();
+        break;
       }
     case TType.MAP:
       {
@@ -69,6 +77,7 @@ public class TProtocolUtil {
           skip(prot, map.valueType);
         }
         prot.readMapEnd();
+        break;
       }
     case TType.SET:
       {        
@@ -77,6 +86,7 @@ public class TProtocolUtil {
           skip(prot, set.elemType);
         }
         prot.readSetEnd();
+        break;
       }
     case TType.LIST:
       {
@@ -85,9 +95,10 @@ public class TProtocolUtil {
           skip(prot, list.elemType);
         }
         prot.readListEnd();
+        break;
       }
     default:
-      return;
+      break;
     }
   }
 }