You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ar...@apache.org on 2007/10/08 16:36:08 UTC

svn commit: r582837 - /incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java

Author: arnaudsimon
Date: Mon Oct  8 07:36:07 2007
New Revision: 582837

URL: http://svn.apache.org/viewvc?rev=582837&view=rev
Log:
fixed acknowledge for not acknowledging messages with a greater tag

Modified:
    incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java

Modified: incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java?rev=582837&r1=582836&r2=582837&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java (original)
+++ incubator/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQSession_0_10.java Mon Oct  8 07:36:07 2007
@@ -158,10 +158,14 @@
         {
             for (Long messageTag : _unacknowledgedMessageTags)
             {
-                ranges.add(messageTag);
+                if( messageTag <= deliveryTag )
+                {
+                    ranges.add(messageTag);
+                    _unacknowledgedMessageTags.remove(messageTag);
+                }
             }
             //empty the list of unack messages
-            _unacknowledgedMessageTags.clear();
+
         }
         else
         {