You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2010/09/15 15:19:45 UTC

svn commit: r997318 - /james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java

Author: norman
Date: Wed Sep 15 13:19:45 2010
New Revision: 997318

URL: http://svn.apache.org/viewvc?rev=997318&view=rev
Log:
Correct parsing of sequence-set / id-ranges (IMAP-212)

Modified:
    james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java

Modified: james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java
URL: http://svn.apache.org/viewvc/james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java?rev=997318&r1=997317&r2=997318&view=diff
==============================================================================
--- james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java (original)
+++ james/imap/trunk/message/src/main/java/org/apache/james/imap/decode/base/AbstractImapCommandParser.java Wed Sep 15 13:19:45 2010
@@ -608,7 +608,7 @@ public abstract class AbstractImapComman
             	// See https://issues.apache.org/jira/browse/IMAP-212
                 long val1 = parseLong(range.substring(0, pos));
                 long val2 = parseLong(range.substring(pos + 1));
-                if (val1 <= val2) {
+                if (val1 <= val2 || val1 == Long.MAX_VALUE) {
                 	return new IdRange(val1, val2);
                 } else {
                 	return new IdRange(val2, val1);



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org