tracker r2789 - in trunk: . src/tracker-indexer

[carlosg svn gnome org]

Author: carlosg
Date: Tue Jan 13 14:44:51 2009
New Revision: 2789
URL: http://svn.gnome.org/viewvc/tracker?rev=2789&view=rev

Log:
2009-01-13  Carlos Garnacho  <carlos imendio com>

        * src/tracker-indexer/tracker-indexer.c (process_module): Do not spew
        a critical warning if the module has no dirs configured. Do not emit
        module-finished/started signals for misconfigured modules.

Modified:
   trunk/ChangeLog
   trunk/src/tracker-indexer/tracker-indexer.c

Modified: trunk/src/tracker-indexer/tracker-indexer.c
==============================================================================
--- trunk/src/tracker-indexer/tracker-indexer.c	(original)
+++ trunk/src/tracker-indexer/tracker-indexer.c	Tue Jan 13 14:44:51 2009
@@ -2403,9 +2403,6 @@
 
 	module = g_hash_table_lookup (indexer->private->indexer_modules, module_name);
 
-	/* Signal module start/stop */
-	process_module_emit_signals (indexer, module_name);
-
 	if (!module) {
 		/* No need to signal stopped here, we will get that
 		 * signal the next time this function is called.
@@ -2414,10 +2411,16 @@
 		return;
 	}
 
+	dirs = tracker_module_config_get_monitor_recurse_directories (module_name);
+
+	if (!dirs) {
+		return;
+	}
+
 	g_message ("Starting module:'%s'", module_name);
 
-	dirs = tracker_module_config_get_monitor_recurse_directories (module_name);
-	g_return_if_fail (dirs != NULL);
+	/* Signal module start/stop */
+	process_module_emit_signals (indexer, module_name);
 
 	for (d = dirs; d; d = d->next) {
 		PathInfo *info;