Cool! But it doesn't actually do the split.  Fix attached.

On 10/1/06, Jamie McCracken <[EMAIL PROTECTED]> wrote:
Edward Duffy wrote:
> Hey -
>
> What do you think about this patch?  If trackerd finds Doc.Keywords
> embeded in a file, it'll automatically adds those keywords to
> tracker's tag list.  If you like, I can update it for File.Keywords
> and Imag.Keywords.

I added this for Doc.Keywords and Image.Keywords.

I added it in a different place and did it slightly differently.

It needs testing to make sure it handles changed keywords in
Doc.Keywords and Image.Keywords during update.


--
Mr Jamie McCracken
http://jamiemcc.livejournal.com/

Index: src/trackerd/tracker-db.c
===================================================================
RCS file: /cvs/gnome/tracker/src/trackerd/tracker-db.c,v
retrieving revision 1.29
diff -u -p -r1.29 tracker-db.c
--- src/trackerd/tracker-db.c	1 Oct 2006 11:18:48 -0000	1.29
+++ src/trackerd/tracker-db.c	3 Oct 2006 19:11:10 -0000
@@ -215,7 +215,7 @@ tracker_db_add_embedded_keywords (DBConn
 	char **array, **tags;
 	char *tag;
 	
-	array = g_strsplit (keywords, "\t\n\v\f\r !\"#$%&'()*/<=>?[\\]^`{|}~+,.:;@\"[]", -1);
+	array = g_strsplit_set (keywords, "\t\n\v\f\r !\"#$%&'()*/<=>?[\\]^`{|}~+,.:;@\"[]", -1);
 
 	for (tags = array; *tags; ++tags) {
 
_______________________________________________
tracker-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/tracker-list

Reply via email to