Don't create copies of indirectly returned values

This improves performance and more importantly, fixes a memory leak caused by the original value not being destroyed. Signed-off-by: Dimitar Dobrev <dpldobrev@protonmail.com>
mono · Aug 20, 2019 · a417467 · a417467
1 parent 8815307
commit a417467
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/src/Generator/Generators/CSharp/CSharpSources.cs b/src/Generator/Generators/CSharp/CSharpSources.cs
@@ -2808,6 +2808,7 @@ public void GenerateFunctionCall(string functionName, List<Parameter> parameters
 
             var originalFunction = function.OriginalFunction ?? function;
 
+            var names = new List<string>();
             if (originalFunction.HasIndirectReturnTypeParameter)
             {
                 var indirectRetType = originalFunction.Parameters.First(
@@ -2825,8 +2826,8 @@ public void GenerateFunctionCall(string functionName, List<Parameter> parameters
                     Class retClass;
                     type.TryGetClass(out retClass);
                     var @class = retClass.OriginalClass ?? retClass;
-                    WriteLine($@"var {Helpers.ReturnIdentifier} = new {
-                        TypePrinter.PrintNative(@class)}();");
+                    WriteLine($@"var {Helpers.ReturnIdentifier} = Marshal.AllocHGlobal({
+                        @class.Layout.GetSize()});");
                 }
                 else
                 {
@@ -2839,13 +2840,15 @@ public void GenerateFunctionCall(string functionName, List<Parameter> parameters
 
                         WriteLine("{0} {1};", typeMap.CSharpSignatureType(typePrinterContext),
                             Helpers.ReturnIdentifier);
+                        names.Add($"new IntPtr(&{Helpers.ReturnIdentifier})");
                     }
                     else
                         WriteLine("var {0} = {1};", construct);
                 }
+                if (names.Count == 0)
+                    names.Add(Helpers.ReturnIdentifier);
             }
 
-            var names = new List<string>();
             foreach (var param in @params)
             {
                 if (param.Param == operatorParam && needsInstance)
@@ -2862,12 +2865,6 @@ public void GenerateFunctionCall(string functionName, List<Parameter> parameters
 
             var needsFixedThis = needsInstance && isValueType;
 
-            if (originalFunction.HasIndirectReturnTypeParameter)
-            {
-                var name = string.Format("new IntPtr(&{0})", Helpers.ReturnIdentifier);
-                names.Insert(0, name);
-            }
-
             if (needsInstance)
             {
                 var instanceIndex = GetInstanceParamIndex(method);

diff --git a/src/Generator/Types/Std/Stdlib.cs b/src/Generator/Types/Std/Stdlib.cs
@@ -382,8 +382,7 @@ public override void CSharpMarshalToManaged(CSharpMarshalContext ctx)
                 ctx.MarshalKind == MarshalKind.ReturnVariableArray;
             ctx.Before.WriteLine($@"var {varBasicString} = {
                 basicString.Visit(typePrinter)}.{Helpers.CreateInstanceIdentifier}({
-                (usePointer ? string.Empty : $"new {typePrinter.IntPtrType}(&")}{
-                 ctx.ReturnVarName}{(usePointer ? string.Empty : ")")});");
+                 ctx.ReturnVarName});");
             string @string = $"{qualifiedBasicString}Extensions.{data.Name}({varBasicString})";
             if (usePointer)
             {