Fixed leaks which were causing generation to use too much memory.

Signed-off-by: Dimitar Dobrev <[email protected]>
ddobrev · Dec 11, 2015 · 9f3d2f6 · 9f3d2f6
1 parent d279d68
commit 9f3d2f6
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 14 deletions.
diff --git a/QtSharp.CLI/Program.cs b/QtSharp.CLI/Program.cs
@@ -85,14 +85,16 @@ public static int Main(string[] args)
             foreach (var libFile in libFiles)
             {
                 parserOptions.FileName = libFile;
-                var parserResult = ClangParser.ParseLibrary(parserOptions);
-                if (parserResult.Kind == ParserResultKind.Success)
+                using (var parserResult = ClangParser.ParseLibrary(parserOptions))
                 {
-                    dependencies[libFile] = CppSharp.ClangParser.ConvertLibrary(parserResult.Library).Dependencies;
-                }
-                else
-                {
-                    dependencies[libFile] = Enumerable.Empty<string>();
+                    if (parserResult.Kind == ParserResultKind.Success)
+                    {
+                        dependencies[libFile] = CppSharp.ClangParser.ConvertLibrary(parserResult.Library).Dependencies;
+                    }
+                    else
+                    {
+                        dependencies[libFile] = Enumerable.Empty<string>();
+                    }
                 }
             }
             var modules = new List<string>

diff --git a/QtSharp/CompileInlinesPass.cs b/QtSharp/CompileInlinesPass.cs
@@ -47,12 +47,14 @@ public override bool VisitLibrary(ASTContext library)
             var parserOptions = new ParserOptions();
             parserOptions.addLibraryDirs(dir);
             parserOptions.FileName = inlines;
-            var parserResult = ClangParser.ParseLibrary(parserOptions);
-            if (parserResult.Kind == ParserResultKind.Success)
+            using (var parserResult = ClangParser.ParseLibrary(parserOptions))
             {
-                var nativeLibrary = CppSharp.ClangParser.ConvertLibrary(parserResult.Library);
-                this.Driver.Symbols.Libraries.Add(nativeLibrary);
-                this.Driver.Symbols.IndexSymbols();
+                if (parserResult.Kind == ParserResultKind.Success)
+                {
+                    var nativeLibrary = CppSharp.ClangParser.ConvertLibrary(parserResult.Library);
+                    this.Driver.Symbols.Libraries.Add(nativeLibrary);
+                    this.Driver.Symbols.IndexSymbols();
+                }
             }
             return true;
         }

diff --git a/QtSharp/Documentation.cs b/QtSharp/Documentation.cs
@@ -277,8 +277,6 @@ public void DocumentProperty(Property property)
 		        var name = macroExpansion.Text.Split(' ')[1];
 		        if (name == property.Name || name == alternativeName)
 		        {
-                    property.LineNumberStart = macroExpansion.LineNumberStart;
-                    property.LineNumberEnd = macroExpansion.LineNumberEnd;
                     property.Name = name;
 					this.DocumentQtProperty(property);
 					return;