Re: [evolution-patches] [Mail]Fix for bug 333243

[jeff cai <Jeff Cai Sun COM>]

Hi, Jeff
I found the real cause that sometimes the url is NULL.
When a new user starts to use evolution, first he will config an account
for evolution. Function emae_commit will be invoked to write
the account information to configuration file. Meanwhile, an "add new
account" request message will be put into a message queue. Then,
evolution will load all accounts in the configuration file as what it
will do whenever it starts later. Note, the account info added just now
has been in that file, so the account will be loaded twice. When it is
loaded a second time, it will remove the account loaded last time whose
folders in tree model has not been created yet. In this case, we will
get an null folder url.

That patch I gave can solve this problem. Do you think it can be
committed into CVS now?

Jeff Cai.
 
On Fri, 2006-03-03 at 10:05 -0500, Jeffrey Stedfast wrote:
> I think the more important question is how is this API getting a NULL
> uri?
> 
> your fix may be correct, but it's still important to know WHY it is
> sometimes NULL first to be sure we aren't just fixing a symptom rather
> than the real bug.
> 
> Jeff
> 
> On Fri, 2006-03-03 at 21:23 +0800, jeff cai wrote:
> > Hi
> > If uri is null, don't remove it.
> > 
> > Please review it.
> > 
> > 
> > _______________________________________________
> > Evolution-patches mailing list
> > Evolution-patches gnome org
> > http://mail.gnome.org/mailman/listinfo/evolution-patches