[cutter-commit 04442] clear-code/cutter@2b66587 [master] Plug memory leaks on loading all modules

[Kouhei S. <nul...@cl...>]

Kouhei Sutou	2014-01-19 16:02:10 +0900 (Sun, 19 Jan 2014)

  New Revision: 2b665870b8ddc5dcd6f7b2cbce956bf9ae450957
  https://github.com/clear-code/cutter/commit/2b665870b8ddc5dcd6f7b2cbce956bf9ae450957

  Message:
    Plug memory leaks on loading all modules

  Modified files:
    cutter/cut-report-factory-builder.c
    cutter/cut-stream-factory-builder.c
    cutter/cut-ui-factory-builder.c

  Modified: cutter/cut-report-factory-builder.c (+3 -0)
===================================================================
--- cutter/cut-report-factory-builder.c    2013-12-26 10:42:39 +0900 (51a4508)
+++ cutter/cut-report-factory-builder.c    2014-01-19 16:02:10 +0900 (43d3490)
@@ -294,6 +294,9 @@ build_all (CutFactoryBuilder *builder)
         factories = g_list_prepend(factories, module_factory);
     }
 
+    g_list_foreach(factory_names, (GFunc)g_free, NULL);
+    g_list_free(factory_names);
+
     return g_list_reverse(factories);
 }
 

  Modified: cutter/cut-stream-factory-builder.c (+3 -0)
===================================================================
--- cutter/cut-stream-factory-builder.c    2013-12-26 10:42:39 +0900 (8871372)
+++ cutter/cut-stream-factory-builder.c    2014-01-19 16:02:10 +0900 (5b481a8)
@@ -248,6 +248,9 @@ build_all (CutFactoryBuilder *builder)
         factories = g_list_prepend(factories, module_factory);
     }
 
+    g_list_foreach(factory_names, (GFunc)g_free, NULL);
+    g_list_free(factory_names);
+
     return g_list_reverse(factories);
 }
 

  Modified: cutter/cut-ui-factory-builder.c (+3 -0)
===================================================================
--- cutter/cut-ui-factory-builder.c    2013-12-26 10:42:39 +0900 (e4c7f5c)
+++ cutter/cut-ui-factory-builder.c    2014-01-19 16:02:10 +0900 (e170107)
@@ -188,6 +188,9 @@ build_all (CutFactoryBuilder *builder)
         factories = g_list_prepend(factories, module_factory);
     }
 
+    g_list_foreach(factory_names, (GFunc)g_free, NULL);
+    g_list_free(factory_names);
+
     return g_list_reverse(factories);
 }