You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by ju...@apache.org on 2006/08/07 08:35:18 UTC

svn commit: r429253 - /jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java

Author: jukka
Date: Sun Aug  6 23:35:18 2006
New Revision: 429253

URL: http://svn.apache.org/viewvc?rev=429253&view=rev
Log:
JCR-532: Correctly update the positionOfBuffer variable when skipping many items.

Modified:
    jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java

Modified: jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java?rev=429253&r1=429252&r2=429253&view=diff
==============================================================================
--- jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java (original)
+++ jackrabbit/trunk/jcr-rmi/src/java/org/apache/jackrabbit/rmi/client/iterator/ClientIterator.java Sun Aug  6 23:35:18 2006
@@ -129,8 +129,10 @@
             positionInBuffer += skipNum;
         } else {
             try {
-                remote.skip(skipNum - (buffer.length - positionInBuffer));
+                skipNum -= buffer.length - positionInBuffer;
+                remote.skip(skipNum);
                 positionInBuffer = buffer.length;
+                positionOfBuffer += skipNum;
             } catch (RemoteException e) {
                 throw new RemoteRuntimeException(e);
             } catch (NoSuchElementException e) {