In the patched version, the call to mail_check() is moved to inside the idle loop, right after the ping_mailbox() call, instead of being called just once at the the start of IDLE processing. This causes at least one additional burp attempt, perhaps two more, while IDLE is in process.
In the imap-2007b/c code, the call to mail_check() is outside the do-loop. Assuming that the underlying problem -- mailboxes not being burped -- is due to there being a sharer when the burp was attempted, and that there are no sharers during the 30 minutes in the idle loop, this patch should remediate that underlying problem. -- Mark -- http://panda.com/mrc Democracy is two wolves and a sheep deciding what to eat for lunch. Liberty is a well-armed sheep contesting the vote. > Date: Thu, 18 Sep 2008 13:44:42 -0400 > From: [EMAIL PROTECTED] > To: [EMAIL PROTECTED] > CC: [email protected] > Subject: RE: [Imap-uw] RIM Blackberry BIS service and inability to reclaim > space (burping) following delete/expunge > > Thank you for the updated patch, it is greatly appreciated! > > I tried patching and testing, but it appears the code segment that > is attached is the same as that found in the imap-2007b distribution, > with the exception of 7 lines which have tab/space differences. > > In case I was missing something, I built using this code segment but > saw no change from the original distribution. Is there any chance > you sent the wrong code segment? > > -Dave > > >> Date: Thu, 18 Sep 2008 11:52:42 >> From: Mark Crispin >> To: Dave Halsema , [email protected] >> Subject: RE: [Imap-uw] RIM Blackberry BIS service and inability to reclaim >> space (burping) following delete/expunge >> >> >> Two other people reported the same issue. I suspect that the problem >> is the unguarded fs_give(). >> >> Try the attached more detailed patch, which is what I actually did as >> opposed to what I typed in off-hand into Hotmail. It replaces the >> existing IDLE code in imapd.c; delete the old code and drop the new >> code in. Make sure that you get the correct places to delete/insert. >> >> I would like definite confirmation that it works. >> >> -- Mark -- >> >> http://panda.com/mrc >> Democracy is two wolves and a sheep deciding what to eat for lunch. >> Liberty is a well-armed sheep contesting the vote. >> >> _________________________________________________________________ >> Get more out of the Web. Learn 10 hidden secrets of Windows Live. >> http://windowslive.com/connect/post/jamiethomson.spaces.live.com-Blog-cns!550F681DAD532637!5295.entry?ocid=TXT_TAGLM_WL_domore_092008 _________________________________________________________________ Get more out of the Web. Learn 10 hidden secrets of Windows Live. http://windowslive.com/connect/post/jamiethomson.spaces.live.com-Blog-cns!550F681DAD532637!5295.entry?ocid=TXT_TAGLM_WL_domore_092008_______________________________________________ Imap-uw mailing list [email protected] http://mailman2.u.washington.edu/mailman/listinfo/imap-uw
