Re: Patch: fix concurrent accesses to the imap summary

[Philip Van Hoof <spam pvanhoof be>]

Hmm ic, after some more discussion on IRC I accepted the temporary
solution and I have committed this.

We should find a better way than passing this as a url-string parameter
though.

On Thu, 2008-11-20 at 14:12 +0100, Martin Bonnin wrote:
> Hello Philip,
> 
>   I'm not doing anything illegal with the sessions. Let me explain
> with more details. What I do is basically the following :
> 
>   1.tny_folder_get_msg()
>   2.tny_folder_remove_msg()
>   3.tny_folder_sync()
>   4.tny_folder_get_msg()
> 
>   The server is a Courier server. The above sequence triggers the following :
> 
>   1. tny_folder_get_msg()
> 
> Opens the 2nd session, "Get-Message service created"
> 
>   2. tny_folder_remove_msg()
> 
> sets some flags locally I guess
> 
>   3. tny_folder_sync()
> 
> (48, 48) -> C00027 UID STORE 1009 +FLAGS.SILENT (\Deleted)
> (.., ..) <- C00027 OK STORE completed.
> camel_imap_folder_stop_idle
> (25, 25) -> C00028 UID EXPUNGE 1009
> (.., ..) <- * 7 EXPUNGE
> (.., ..) <- * 17 EXISTS
> (.., ..) <- * 0 RECENT
> (.., ..) <- C00028 OK EXPUNGE completed
> 
> so here *message 7* is removed from the summary
> 
>   4.tny_folder_get_msg()
> 
> Get-Message service reused
> (13, 13) -> D00007 NOOP
> (.., ..) <- * 7 EXPUNGE
> (.., ..) <- * 17 EXISTS
> (.., ..) <- * 0 RECENT
> (.., ..) <- D00007 OK NOOP completed
> ...
> 
> so here *message 7* is removed from the summary a second time but is
> not valid anymore :-(
> 
> Do I miss something stupid ?
> 
-- 
Philip Van Hoof, freelance software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://pvanhoof.be/blog
http://codeminded.be