[sup-devel] [PATCH] fix crash during poll if Maildir has been emptied

From: Sascha Silbe <sascha-pgp@silbe.org>
To: sup-devel <sup-devel@rubyforge.org>
Subject: [sup-devel] [PATCH] fix crash during poll if Maildir has been emptied
Date: Thu,  8 Jul 2010 15:26:37 +0000	[thread overview]
Message-ID: <1278602797-2553-1-git-send-email-sascha-pgp@silbe.org> (raw)

We need to do the rescan in start_offset instead of end_offset so that
end_offset won't be called if the Maildir is empty after the rescan.

Fixes this crash:

NoMethodError from thread: user-invoked poll
undefined method `+' for nil:NilClass
/home/sascha.silbe/src/sup/lib/sup/maildir.rb:171:in `end_offset'
/home/sascha.silbe/src/sup/lib/sup/source.rb:90:in `done?'
[...]

Signed-off-by: Sascha Silbe <sascha-pgp@silbe.org>
---
 lib/sup/maildir.rb |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/sup/maildir.rb b/lib/sup/maildir.rb
index ef3b318..0b2fd5e 100644
--- a/lib/sup/maildir.rb
+++ b/lib/sup/maildir.rb
@@ -162,12 +162,12 @@ class Maildir < Source
   end
 
   def start_offset
-    scan_mailbox
+    scan_mailbox :rescan => true
     @ids.first
   end
 
   def end_offset
-    scan_mailbox :rescan => true
+    scan_mailbox
     @ids.last + 1
   end
 
-- 
1.7.1

_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel

