Fix LightExpression LoopExpression.Type to return BreakLabel type, matching System.Linq.Expressions behavior

Copilot · dadhi · web-flow · commit d6c559a13a78 · 2026-04-08T08:28:38.000Z
Agent-Logs-Url: https://github.com/dadhi/FastExpressionCompiler/sessions/a43bd802-b944-48bc-8c8c-c0abc89bc6e6 Co-authored-by: dadhi <39516+dadhi@users.noreply.github.com>
diff --git a/src/FastExpressionCompiler.LightExpression/Expression.cs b/src/FastExpressionCompiler.LightExpression/Expression.cs
@@ -5160,7 +5160,7 @@ internal TypedManyVariablesBlockExpression(Type type, IReadOnlyList<ParameterExp
 public sealed class LoopExpression : Expression
 {
     public override ExpressionType NodeType => ExpressionType.Loop;
-    public override Type Type => typeof(void);
+    public override Type Type => BreakLabel?.Type ?? typeof(void);
     public readonly Expression Body;
     public readonly LabelTarget BreakLabel;
     public readonly LabelTarget ContinueLabel;

Original file line number	Diff line number	Diff line change
`@@ -5160,7 +5160,7 @@ internal TypedManyVariablesBlockExpression(Type type, IReadOnlyList<ParameterExp`
`5160`	`5160`	`public sealed class LoopExpression : Expression`
`5161`	`5161`	`{`
`5162`	`5162`	`public override ExpressionType NodeType => ExpressionType.Loop;`
`5163`		`- public override Type Type => typeof(void);`
	`5163`	`+ public override Type Type => BreakLabel?.Type ?? typeof(void);`
`5164`	`5164`	`public readonly Expression Body;`
`5165`	`5165`	`public readonly LabelTarget BreakLabel;`
`5166`	`5166`	`public readonly LabelTarget ContinueLabel;`