Re: [Tracker] Indexing mbox files

2010-08-23 Thread Philip Van Hoof
On Wed, 2010-08-18 at 18:36 +0200, Mildred Ki'Lya wrote: On 2010-08-18 18:16, Mildred Ki'Lya wrote: So, if I understood everything well, I should have something like: nmo:Mailbox - nie:hasPart - nmo:MailboxDataObject - nie:interpretedAs - nmo:Email and of course nmo:Email -

[Tracker] Review of tracker-writeback migration to libtracker-sparql

2010-08-23 Thread Martyn Russell
This is the review of commit: http://git.gnome.org/browse/tracker/commit/?h=class-signalid=a28504d1aa23cbf8957d2f1adf66694e006dad0e 1. sparql_query_cb() checks error twice: i.e. if (!error) { .. } else { if (error != NULL) { } else { } } 2. For the playlist setting, you use:

Re: [Tracker] Review of tracker-writeback migration to libtracker-sparql

2010-08-23 Thread Philip Van Hoof
Fixes for these issues: http://git.gnome.org/browse/tracker/commit/?h=class-signalid=cee62fb31d8d9a0f5aef5773247d2f3c9818c977 On Mon, 2010-08-23 at 17:10 +0100, Martyn Russell wrote: This is the review of commit: