Author: vmpn
Date: Fri May 25 15:19:21 2012
New Revision: 1342682

URL: http://svn.apache.org/viewvc?rev=1342682&view=rev
Log:
JavaHL: Changed return value from the java svn_stream_t read function to be 
compatible with the txdelta_next_window function

[ in subversion/bindings/javahl/native ]

* InputStream.cpp
  (read): Return 0 instead of -1 as expected by the txdelta_next_window function

Modified:
    
subversion/branches/javahl-ra/subversion/bindings/javahl/native/InputStream.cpp

Modified: 
subversion/branches/javahl-ra/subversion/bindings/javahl/native/InputStream.cpp
URL: 
http://svn.apache.org/viewvc/subversion/branches/javahl-ra/subversion/bindings/javahl/native/InputStream.cpp?rev=1342682&r1=1342681&r2=1342682&view=diff
==============================================================================
--- 
subversion/branches/javahl-ra/subversion/bindings/javahl/native/InputStream.cpp 
(original)
+++ 
subversion/branches/javahl-ra/subversion/bindings/javahl/native/InputStream.cpp 
Fri May 25 15:19:21 2012
@@ -99,6 +99,14 @@ svn_error_t *InputStream::read(void *bat
   if (JNIUtil::isJavaExceptionThrown())
     return SVN_NO_ERROR;
 
+  /*
+   * Convert -1 from InputStream.read that means EOF, 0 which is subversion 
equivalent
+   */
+  if(jread == -1)
+    {
+      jread = 0;
+    }
+
   // Put the Java byte array into a helper object to retrieve the
   // data bytes.
   JNIByteArray outdata(data, true);
@@ -107,7 +115,7 @@ svn_error_t *InputStream::read(void *bat
 
   // Catch when the Java method tells us it read too much data.
   if (jread > (jint) *len)
-    jread = -1;
+    jread = 0;
 
   // In the case of success copy the data back to the Subversion
   // buffer.


Reply via email to