Re: [evolution-patches] Patch for bug #127541 - Default Free Busy URI

[Harish Krishnaswamy <kharish novell com>]

hi,

The async f/b code looks ok to me.
Thanks for correcting the bug in the refresh_data clean up :)

wanted to try out retrieving the f/b (the main patch) too
- jp gave me some test urls that you had set up.. couldn't access
them :(
hope to catch u on irc sometime for some help on setting it up and
seeing it live.

Sorry for not looking into this sooner..

thanks,
harish

On Thu, 2004-12-02 at 11:05 -0400, James Bowes wrote:
> On Wed, 2004-01-12 at 09:50 -0500, JP Rosevear wrote:
> > Ok, a few more comments.  You are using %d and %u now but the schema
> > default has not changed.  In fact, I'm not sure there should be a
> > default, because if someone never sets it they will get a lot of useless
> > lookups and anger admins whose servers are being pinged.  The focus
> > stuff does cut down on f/b lookups when a meeting is open and the url is
> > changing but doesn't really mesh with how the other settings are set.
> > Perhaps calendar-config or e-meeting-time-sel could wait 5 seconds or
> > something before starting a refresh when the url changes? And reset to 5
> > seconds if it changes in the 5 second interval.
> > 
> 
> I got rid of the default in the schema, and set up a 5 second g_timeout
> for the refresh (in e-meeting-time-sel). If the source function for the
> timeout has not been called when there is a change in the gconf setting,
> It will be removed, and a new one will be created. 
> 
> > I'll let harish comment on the async loading stuff since that's his
> > code.
> > 
> > Looking pretty sweet right now.
> > 
> 
> Thanks :)
> 
> > -JP
> 
> James