CSharpExpressionPrinter: Wrap expression in parenthesis

Wrap default parameter expressions in parentheses to ensure whole expressions are casted instead of only the first operand.
mono · Jul 9, 2023 · 26e4984 · 26e4984
1 parent 9923d79
commit 26e4984
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Generator/Generators/CSharp/CSharpExpressionPrinter.cs b/src/Generator/Generators/CSharp/CSharpExpressionPrinter.cs
@@ -27,11 +27,11 @@ public string VisitParameter(Parameter parameter)
             if (desugared.IsPrimitiveType() &&
                 (parameter.DefaultArgument.Declaration != null ||
                  parameter.DefaultArgument.Class == StatementClass.BinaryOperator))
-                return $"({desugared.Visit(typePrinter)}) {expression}";
+                return $"({desugared.Visit(typePrinter)}) ({expression})";
             var finalType = (desugared.GetFinalPointee() ?? desugared).Desugar();
             if (finalType.TryGetClass(out var @class) && @class.IsInterface)
                 return $@"({@class.Visit(typePrinter)}) ({
-                    @class.OriginalClass.Visit(typePrinter)}) {expression}";
+                    @class.OriginalClass.Visit(typePrinter)}) ({expression})";
             return expression;
         }
 
@@ -88,4 +88,4 @@ public string ToString(Type type)
 
         private readonly TypePrinter typePrinter;
     }
-}
+}