Philip Van Hoof wrote:
On Thu, 2009-04-02 at 14:23 +0200, Philip Van Hoof wrote:
I'm not very sure, but isn't this going to make things faster?

Let me know ...

Added a break, don't know why we would need to continue looping once
found.



Hi Philip,

Superb patch, thanks. What I love is, no memory creation/freeing takes place with lists and you are right, we were traversing the list twice it seems with the list and the keys->list.

Just as a note, I changed the code slightly so we return NULL at the end, return the module name in the inner most block and moved some variable declarations to the right scope (that was there before though). The casting there wasn't necessary (const gchar*) and (GHashTable*).

Thanks again, committed!

--
Regards,
Martyn
_______________________________________________
tracker-list mailing list
tracker-list@gnome.org
http://mail.gnome.org/mailman/listinfo/tracker-list

Reply via email to