add check for AllowPrintIL to #478

Author: dadhi

src/FastExpressionCompiler/TestTools.cs

@@ -154,6 +154,8 @@ public static void PrintIL(this MethodInfo method, [CallerMemberName] string tag
 
     public static void PrintIL(this IDelegateDebugInfo diagInfo, [CallerMemberName] string tag = null)
     {
+        if (!AllowPrintIL) return;
+
         SmallMap4<IDelegateDebugInfo, string, RefEq<IDelegateDebugInfo>,
             SmallMap4.SingleArrayEntries<IDelegateDebugInfo, string, RefEq<IDelegateDebugInfo>>
         > uniquePrinted = default;

test/FastExpressionCompiler.IssueTests/Issue347_InvalidProgramException_on_compiling_an_expression_that_returns_a_record_which_implements_IList.cs

@@ -83,24 +83,22 @@ public void Test_struct_parameter_in_closure_of_the_nested_lambda()
 
             var f = expr.CompileFast(true, CompilerFlags.EnableDelegateDebugInfo);
             Asserts.IsNotNull(f);
-            f.PrintIL();
 
-            var di = f.TryGetDebugInfo();
-            foreach (var ni in di.EnumerateNestedLambdas())
-            {
-                ni.PrintIL("nested");
-                ni.AssertOpCodes(
-                    OpCodes.Ldarg_0,
-                    OpCodes.Ldfld,      // ArrayClosureWithNonPassedParams.NonPassedParams
-                    OpCodes.Ldc_I4_0,
-                    OpCodes.Ldelem_Ref,
-                    OpCodes.Unbox_Any,  // NotifyModel
-                    OpCodes.Stloc_0,
-                    OpCodes.Ldloca_S,   // 0
-                    OpCodes.Call,       // NotifyModel.get_Number1
-                    OpCodes.Ret
-                );
-            }
+            var fd = f.TryGetDebugInfo();
+            fd.PrintIL();
+
+            var nested = fd.EnumerateNestedLambdas().First();
+            nested.AssertOpCodes(
+                OpCodes.Ldarg_0,
+                OpCodes.Ldfld,      // ArrayClosureWithNonPassedParams.NonPassedParams
+                OpCodes.Ldc_I4_0,
+                OpCodes.Ldelem_Ref,
+                OpCodes.Unbox_Any,  // NotifyModel
+                OpCodes.Stloc_0,
+                OpCodes.Ldloca_S,   // 0
+                OpCodes.Call,       // NotifyModel.get_Number1
+                OpCodes.Ret
+            );
 
             var y = f(m);
             Asserts.AreEqual(43, y);

test/FastExpressionCompiler.TestsRunner/Program.cs

@@ -16,10 +16,10 @@ public static void Main()
 
             new LightExpression.UnitTests.NestedLambdasSharedToExpressionCodeStringTest().Run();
 
+            new LightExpression.IssueTests.Issue347_InvalidProgramException_on_compiling_an_expression_that_returns_a_record_which_implements_IList().Run();
             // new Issue316_in_parameter().Run();
             // new LightExpression.IssueTests.Issue55_CompileFast_crash_with_ref_parameter().Run();
             // new LightExpression.IssueTests.Issue341_Equality_comparison_between_nullable_and_null_inside_Any_produces_incorrect_compiled_expression().Run();
-            // new LightExpression.IssueTests.Issue347_InvalidProgramException_on_compiling_an_expression_that_returns_a_record_which_implements_IList().Run();
             // new Issue441_Fails_to_pass_Constant_as_call_parameter_by_reference().Run();
             // new Issue461_InvalidProgramException_when_null_checking_type_by_ref().Run();