Re: [Tracker] Please review, thumbnail cleanup code

2008-11-25 Thread Philip Van Hoof
This was reposted using the e-mail address I use for this mailing list, no need to review (it's already committed) On Fri, 2008-11-21 at 16:44 +0100, Philip Van Hoof wrote: > Please review this patch > > ___ > tracker-list mailing list > tracker-list@gn

[Tracker] Please review, thumbnail cleanup code

2008-11-25 Thread Philip Van Hoof
Please review this patch -- Philip Van Hoof freelance software developer Codeminded BVBA - http://codeminded.be Index: src/tracker-indexer/tracker-thumbnailer.c === --- src/tracker-indexer/tracker-thumbnailer.c (revision 0) +++ src

Re: [Tracker] Please review, thumbnail cleanup code

2008-11-24 Thread Philip Van Hoof
Thanks for the review and path improvements, will be committing this in a minute. On Mon, 2008-11-24 at 12:44 +, Martyn Russell wrote: > On 21/11/08 15:46, Philip Van Hoof wrote: > > Please review this patch > > Hi Phillip, > > I have attached an update to your patch. I have changed: > > *

Re: [Tracker] Please review, thumbnail cleanup code

2008-11-24 Thread Martyn Russell
On 21/11/08 15:46, Philip Van Hoof wrote: Please review this patch Hi Phillip, I have attached an update to your patch. I have changed: * Make use of GStaticPrivate for the static module data in the tracker-thumbnailer module. * Added an init() and shutdown() function for the tracker-main.

[Tracker] Please review, thumbnail cleanup code

2008-11-21 Thread Philip Van Hoof
Please review this patch -- Philip Van Hoof freelance software developer Codeminded BVBA - http://codeminded.be -- 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 Index: src/tracker-indexe