From 780e4ecac988ba8aeea794631da2ea9a24c7717b Mon Sep 17 00:00:00 2001
From: Mark Slee <mcslee@apache.org>
Date: Tue, 2 Oct 2007 04:43:31 +0000
Subject: [PATCH] Fix missing break statements in Java ProtocolUtil

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 | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/lib/java/src/protocol/TProtocolUtil.java b/lib/java/src/protocol/TProtocolUtil.java
index 935163c8..ad333bc2 100644
--- a/lib/java/src/protocol/TProtocolUtil.java
+++ b/lib/java/src/protocol/TProtocolUtil.java
@@ -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;
     }
   }
 }
-- 
2.17.1