Re: Memory corrections break pop support

[Philip Van Hoof <spam pvanhoof be>]

On Mon, 2006-06-05 at 12:50 +0200, Philip Van Hoof wrote:
> The corrections are right, but it looks like it broke support for pop
> boxes. Or, for any folder type that doesn't support summaries.
> 
> I'm also finding lots of camel issues after fixing some reference
> counting problems in tinymail.
> 
> Well, it still works somewhat, but the message-instance is now really
> removed (and no longer cached in the folder either). The pop-version of
> the message header didn't survive those changes.
> 
> I'm rather going to investigate how evolution is doing pop folders.
> Because frankly, the support was flawed anyway (it didn't use the
> summary information, it uses the full message each-time you want a
> header -- which is totally overkill --).
> 
> So .. If somebody is into it, redoing the pop-part is something I could
> use help with :)

Evolution uses the CamelFilterDriver for this: They don't "show" pop
accounts. They only "show" local folders (mbox folders). The
CamelFilterDriver is utilised for after opening a pop-folder, flowing
messages through it. The filterdriver instance will "filter" the
messages into one of those mbox folders. The folder-view of Evolution
doesn't "show" the pop folders themselves.

Tinymail, at this moment, *does* show the pop folder itself. Therefore I
had to implement a little hack to get around the
camel/providers/pop3-has-no-summaries support. That has no longer works
correctly (it still works but it's crashy at this moment).

I don't think we should continue with trying to support that hack, and
in stead do what Evolution did: flow or copy the messages to a local
mbox file, and use a local folder (which has summaries support in camel)
for displaying pop accounts.

I do know, and understand, this means that quite a lot disk-space will
be consumed for large pop boxes. The current technique of tinymail
doesn't need a lot

Also note that, this way, evolution keeps a copy of the message twice.
Once in the Inbox mbox file, and once in the directory 
	~/.evolution/mail/pop/$ACCOUNT/cache

Which is, of course, unacceptable on a mobile device :-\


So ... another option is to implement support for summaries for pop3
providers in camel.


-- 
Philip Van Hoof, software developer at x-tend 
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
work: vanhoof at x-tend dot be 
http://www.pvanhoof.be - http://www.x-tend.be