Re: [Tracker] ANNOUNCE: tracker 0.6.91 released

[Martyn Russell <martyn imendio com>]

Jamie McCracken wrote:
> On Mon, 2009-03-16 at 11:26 +0200, Ivan Frade wrote:
> > Hi!
> >
> > On Mon, Mar 16, 2009 at 11:02 AM, Martyn Russell <martyn imendio com>
> > wrote:
> >         Jamie McCracken wrote:
> >                 
> >                 Anyway I think thats better than say calc stats
> >                 continuously especially
> >                 as applet requests stats every few seconds if you have
> >                 the stat window
> >                 open
> >         
> >         Eeek, why?
> >
> >
> >         
> >         We have a signal which updates the stats when they change. Why
> >         would we want to get the stats every few seconds? Makes more
> >         sense to just update the ones that changed and only request
> >         the stats once.
> >
> >  Historical reasons: We added the signal in tracker 0.6.90, but nobody
> > updated the applet to use it :)

Good point Ivan, I forgot about that.

> well it uses the IndexProgress signal to refresh stats (which means it
> recalcs them every time which is often every few seconds)

That doesn't have any statistical information about the categories 
though of course and it doesn't make sense to get the statistics every 
few seconds for a few reasons:

1. We have to pause the indexer when we put a request through.
2. We already update when we load the dialog.
3. We now update when we get "finished" for each module.
4. We now have a refresh button to allow the user to explicitly refresh.

> So I guess that needs changing then if you dont want to change things in
> the daemon

The daemon was broken I found out today. The signal we were using was 
not the same as the signal we publish in the .xml DBus introspection 
file. I have fixed this now.

--
Regards,
Martyn