Is this change specific to the branch, or is is beneficial for general JavaHL use? If the latter, it should be committed to trunk first, then backported to the branch.
I realize you don't (yet) have commit privileges to trunk. The way this would usually work is that you'd post the patch, somebody would either apply it to trunk, or just ask you to apply it, and you'd reference that approval in the commit message. -Hyrum On Fri, May 25, 2012 at 10:19 AM, <v...@apache.org> wrote: > 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. > > -- uberSVN: Apache Subversion Made Easy http://www.uberSVN.com/
