[tracker] Code cleanup so we can remove a condition.



commit bb6d363f2973dc8d1a6bcd0d0fa3b09fa7357dcb
Author: Carlos Garnacho <carlos lanedo com>
Date:   Wed Jul 29 17:38:59 2009 +0200

    Code cleanup so we can remove a condition.

 src/tracker-miner-fs/tracker-indexer.c |   17 +++++++----------
 1 files changed, 7 insertions(+), 10 deletions(-)
---
diff --git a/src/tracker-miner-fs/tracker-indexer.c b/src/tracker-miner-fs/tracker-indexer.c
index 07226e0..527c722 100644
--- a/src/tracker-miner-fs/tracker-indexer.c
+++ b/src/tracker-miner-fs/tracker-indexer.c
@@ -937,7 +937,7 @@ init_mount_points (TrackerIndexer *indexer)
 	GPtrArray *sparql_result;
 	GError *error = NULL;
 	GString *accumulator;
-	gint i;
+	gint i, state;
 #ifdef HAVE_HAL
 	GList *udis, *u;
 #endif
@@ -961,24 +961,22 @@ init_mount_points (TrackerIndexer *indexer)
 
 	for (i = 0; i < sparql_result->len; i++) {
 		gchar **row;
-		gint state;
 
 		row = g_ptr_array_index (sparql_result, i);
 		state = VOLUME_MOUNTED_IN_STORE;
 
-		if (strcmp (row[0], TRACKER_NON_REMOVABLE_MEDIA_DATASOURCE_URN) == 0) {
-			/* Report non-removable media to be mounted by HAL as well */
-			state |= VOLUME_MOUNTED;
-		}
-
 		g_hash_table_insert (volumes, g_strdup (row[0]), GINT_TO_POINTER (state));
 	}
 
 	g_ptr_array_foreach (sparql_result, (GFunc) g_strfreev, NULL);
 	g_ptr_array_free (sparql_result, TRUE);
 
-	g_hash_table_replace (volumes, g_strdup (TRACKER_NON_REMOVABLE_MEDIA_DATASOURCE_URN),
-	                      GINT_TO_POINTER (VOLUME_MOUNTED));
+	/* Report non-removable media to be mounted by HAL, so it
+	 * gets added on first index, and ignored in subsequent ones
+	 */
+	state = GPOINTER_TO_INT (g_hash_table_lookup (volumes, TRACKER_NON_REMOVABLE_MEDIA_DATASOURCE_URN));
+	state |= VOLUME_MOUNTED;
+	g_hash_table_replace (volumes, g_strdup (TRACKER_NON_REMOVABLE_MEDIA_DATASOURCE_URN), GINT_TO_POINTER (state));
 
 #ifdef HAVE_HAL
 	udis = tracker_storage_get_removable_device_udis (indexer->private->storage);
@@ -987,7 +985,6 @@ init_mount_points (TrackerIndexer *indexer)
 	for (u = udis; u; u = u->next) {
 		const gchar *udi;
 		gchar *removable_device_urn;
-		gint state;
 
 		udi = u->data;
 		removable_device_urn = g_strdup_printf (TRACKER_DATASOURCE_URN_PREFIX "%s", udi);



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]