]> git.donarmstrong.com Git - lilypond.git/commitdiff
plug memleak in case of failed files.
authorHan-Wen Nienhuys <hanwen@xs4all.nl>
Thu, 2 Nov 2006 13:09:02 +0000 (14:09 +0100)
committerHan-Wen Nienhuys <hanwen@xs4all.nl>
Thu, 2 Nov 2006 13:09:02 +0000 (14:09 +0100)
lily/lily-parser-scheme.cc

index 90d9a60366e018a21fc845de9bf91ae55419927d..3092c6ca9d610d32e4c8a2da18a2222bdecdee94 100644 (file)
@@ -110,11 +110,12 @@ LY_DEFINE (ly_parse_file, "ly:parse-file",
       exit (2);
     }
 
+
+  bool error = false;
   if ((file_name != "-") && file_name.empty ())
     {
       warning (_f ("can't find file: `%s'", file));
-      scm_throw (ly_symbol2scm ("ly-file-failed"),
-                scm_list_1 (scm_makfrom0str (file_name.c_str ())));
+      error = true;
     }
   else
     {
@@ -129,14 +130,19 @@ LY_DEFINE (ly_parse_file, "ly:parse-file",
 
       parser->parse_file (init, file_name, out_file);
 
-      bool error = parser->error_level_;
+      error = parser->error_level_;
 
       parser->unprotect ();
-      if (error)
-       /* TODO: pass renamed input file too.  */
-       scm_throw (ly_symbol2scm ("ly-file-failed"),
-                  scm_list_1 (scm_makfrom0str (file_name.c_str ())));
     }
+
+  /*
+    outside the if-else to ensure cleanup fo Sources object, 
+   */
+  if (error)
+    /* TODO: pass renamed input file too.  */
+    scm_throw (ly_symbol2scm ("ly-file-failed"),
+              scm_list_1 (scm_makfrom0str (file_name.c_str ())));
+  
   return SCM_UNSPECIFIED;
 }