Fix debug assertion in ComprehensionScope.GetParentClosureTuple (#1149)

BCSharp · web-flow · commit 2e5e3bcd1c19 · 2021-03-26T14:57:06.000-07:00
diff --git a/Src/IronPython/Compiler/Ast/Comprehension.cs b/Src/IronPython/Compiler/Ast/Comprehension.cs
@@ -246,7 +246,7 @@ internal override bool ExposesLocalVariable(PythonVariable variable) {
         }
 
         internal override MSAst.Expression/*!*/ GetParentClosureTuple() {
-            Debug.Assert(NeedsLocalContext);
+            Debug.Assert(NeedsLocalContext || IsClosure);
             return MSAst.Expression.Call(null, typeof(PythonOps).GetMethod(nameof(PythonOps.GetClosureTupleFromContext)), Parent.LocalContext);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ internal override bool ExposesLocalVariable(PythonVariable variable) {`
`246`	`246`	`}`
`247`	`247`
`248`	`248`	`internal override MSAst.Expression/!/ GetParentClosureTuple() {`
`249`		`- Debug.Assert(NeedsLocalContext);`
	`249`	`+ Debug.Assert(NeedsLocalContext \|\| IsClosure);`
`250`	`250`	`return MSAst.Expression.Call(null, typeof(PythonOps).GetMethod(nameof(PythonOps.GetClosureTupleFromContext)), Parent.LocalContext);`
`251`	`251`	`}`
`252`	`252`