Author: bryanduxbury
Date: Wed Nov  4 00:30:10 2009
New Revision: 832633

URL: http://svn.apache.org/viewvc?rev=832633&view=rev
Log:
THRIFT-620. java: Compact Protocol should call readAll, not read
This patch switches to calling readAll instead of read.

Modified:
    
incubator/thrift/trunk/lib/java/src/org/apache/thrift/protocol/TCompactProtocol.java

Modified: 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/protocol/TCompactProtocol.java
URL: 
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/java/src/org/apache/thrift/protocol/TCompactProtocol.java?rev=832633&r1=832632&r2=832633&view=diff
==============================================================================
--- 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/protocol/TCompactProtocol.java
 (original)
+++ 
incubator/thrift/trunk/lib/java/src/org/apache/thrift/protocol/TCompactProtocol.java
 Wed Nov  4 00:30:10 2009
@@ -554,7 +554,7 @@
    * Read a single byte off the wire. Nothing interesting here.
    */
   public byte readByte() throws TException {
-    trans_.read(byteRawBuf, 0, 1);
+    trans_.readAll(byteRawBuf, 0, 1);
     return byteRawBuf[0];
   }
 
@@ -584,7 +584,7 @@
    */
   public double readDouble() throws TException {
     byte[] longBits = new byte[8];
-    trans_.read(longBits, 0, 8);
+    trans_.readAll(longBits, 0, 8);
     return Double.longBitsToDouble(bytesToLong(longBits));
   }
 
@@ -607,7 +607,7 @@
     if (length == 0) return new byte[0];
 
     byte[] buf = new byte[length];
-    trans_.read(buf, 0, length);
+    trans_.readAll(buf, 0, length);
     return buf;
   }
 


Reply via email to