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/06/29 19:54:52 UTC

svn commit: r959052 - /james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java

Author: norman
Date: Tue Jun 29 17:54:51 2010
New Revision: 959052

URL: http://svn.apache.org/viewvc?rev=959052&view=rev
Log:
MessageRange.range(..) return noew a MessageRange with type ONE if the from and to uid is the same (IMAP-171)

Modified:
    james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java

Modified: james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java
URL: http://svn.apache.org/viewvc/james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java?rev=959052&r1=959051&r2=959052&view=diff
==============================================================================
--- james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java (original)
+++ james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MessageRange.java Tue Jun 29 17:54:51 2010
@@ -72,6 +72,9 @@ public class MessageRange {
         if (to == Long.MAX_VALUE || to < from) {
             to = NOT_A_UID;
             result = from(from);
+        } else if (from == to){ 
+            // from and to is the same so no need to construct a real range
+            result = one(from);
         } else {
             result = new MessageRange(Type.RANGE, from, to);
         }



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