[tracker/live-config-changes] TrackerMinerFS: Add comment about memory usage for priv->directories.



commit 294669aa9972a73560333a9c775190db7914aa0d
Author: Carlos Garnacho <carlos lanedo com>
Date:   Tue Mar 2 17:58:17 2010 +0100

    TrackerMinerFS: Add comment about memory usage for priv->directories.

 src/libtracker-miner/tracker-miner-fs.c    |    8 +++++---
 src/tracker-miner-fs/tracker-miner-files.c |    1 -
 2 files changed, 5 insertions(+), 4 deletions(-)
---
diff --git a/src/libtracker-miner/tracker-miner-fs.c b/src/libtracker-miner/tracker-miner-fs.c
index 6a43aee..c0a6515 100644
--- a/src/libtracker-miner/tracker-miner-fs.c
+++ b/src/libtracker-miner/tracker-miner-fs.c
@@ -606,9 +606,11 @@ fs_finalize (GObject *object)
 
 	g_free (priv->current_parent_urn);
 
-	if (priv->directories) {
-		g_list_free (priv->directories);
-	}
+	/* Not a leak, contents are actually owned
+	 * by priv->config_directories, being freed
+	 * right below.
+	 */
+	g_list_free (priv->directories);
 
 	if (priv->config_directories) {
 		g_list_foreach (priv->config_directories, (GFunc) directory_data_free, NULL);
diff --git a/src/tracker-miner-fs/tracker-miner-files.c b/src/tracker-miner-fs/tracker-miner-files.c
index 83aee39..3b75b36 100644
--- a/src/tracker-miner-fs/tracker-miner-files.c
+++ b/src/tracker-miner-fs/tracker-miner-files.c
@@ -1122,7 +1122,6 @@ miner_files_force_recheck_idle (gpointer user_data)
 {
 	TrackerMinerFiles *miner_files = user_data;
 
-
 	/* Recheck all directories for compliance with the new config */
 	tracker_miner_fs_force_recheck (TRACKER_MINER_FS (miner_files));
 



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