Re: [sup-devel] Maildir souce id rework

* Re: [sup-devel] Maildir souce id rework
       [not found] <6c0c31751003072119i4c330800s6b508157544165e6@mail.gmail.com>
@ 2010-03-08  5:41 ` Scott Henson
  0 siblings, 0 replies; only message in thread
From: Scott Henson @ 2010-03-08  5:41 UTC (permalink / raw)
  To: sup-devel

[-- Attachment #1.1: Type: text/plain, Size: 1424 bytes --]

On Mon, Mar 8, 2010 at 12:19 AM, Scott Henson <sjh@foolishpride.org> wrote:

> I've created a branch that changes the id that the maildir source returns,
> which is then used as the source_info field inside the index.  It seems to
> work well for me and I'd appreciate some more testing.  This work should
> make the maildir source more robust and more friendly to other clients
> accessing the source without completely throwing sup for a loop.  It will
> also help sync back for maildir (more on this in a second email).
>
> A couple notes.  First, the base source.rb class says that the id returned
> needs to be an integer, but that doesn't seem to be the case.  Also, you'll
> note that start_offset and end_offset are not really telling the truth in my
> version of the maildir source.  This doesn't seem to be a problem as nothing
> seems to be using those two functions.  Second, this source might be
> slightly slower in some places than the original source and faster in
> others.  I think most of the work is done in scan_mailbox and the rest of
> the source should be pretty light.  If anyone has any ideas on how to make
> scan_mailbox lighter, I'd be very interested.  Also, reset might take a
> while since it has to sort the ids array.
>
> Any help in making this better would be much appreciated.  Thanks.
>
>
Probably need to provide a link.
http://github.com/shenson/sup/tree/maildir

-- 
Scott Henson

[-- Attachment #1.2: Type: text/html, Size: 1855 bytes --]

[-- Attachment #2: Type: text/plain, Size: 143 bytes --]

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

^ permalink raw reply	[flat|nested] only message in thread