On Fri, 15 Aug 2003, Andreas Aardal Hanssen wrote:
>>If I delete the bincimap-* files between 3 and 4 then it works.
>Excellent - I'll see if I can track down the bug.

Please see if the patch below works. It certainly does for me. The UID
COPY command didn't recognize the UID property, so when Mozilla copied
these messages (UID COPY 3 destination/) it didn't find it, because with
only one message in the mailbox there would be none with sequencenumber 3.

This is a significant change, so when I solve the compile problems that
have been reported its won't be long until I do a new release.

Thanks Jon! :-)

Andy

diff -u -r1.31 -r1.32
--- src/operator-copy.cc        14 Aug 2003 17:23:25 -0000      1.31
+++ src/operator-copy.cc        15 Aug 2003 15:37:59 -0000      1.32
@@ -88,12 +88,12 @@
     return NO;
   }

+  unsigned int mode = Mailbox::SKIP_EXPUNGED;
+  mode |= command.getUidMode() ? Mailbox::UID_MODE : Mailbox::SQNR_MODE;
+
   // Copy each message in the sequence set to the destination mailbox.
   bool success = true;
-  Mailbox::iterator i
-    = srcMailbox->begin(command.bset,
-                       Mailbox::SKIP_EXPUNGED | Mailbox::SQNR_MODE);
-
+  Mailbox::iterator i = srcMailbox->begin(command.bset, mode);
   for (; success && i != srcMailbox->end(); ++i) {
     Message &source = *i;


Reply via email to