Skip to content

Commit

Permalink
Disable Linq.Expression preferInterpretation for bindings
Browse files Browse the repository at this point in the history
The interpreter is broken, it causes problems in #1742.
The bug is reported in dotnet, without any response yet:
dotnet/runtime#96385
  • Loading branch information
exyi committed Jan 27, 2024
1 parent cb8e675 commit 4194d11
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@ public DefaultExpressionToDelegateCompiler(DotvvmConfiguration config)
interpret = config.Debug;
}
public Delegate Compile(LambdaExpression expression) =>
interpret ? expression.Compile(preferInterpretation: interpret) :
// the interpreter is broken: https://github.com/dotnet/runtime/issues/96385
// interpret ? expression.Compile(preferInterpretation: interpret) :
expression.Compile();
// TODO: use FastExpressionCompiler
// we can't do that atm since it still has some bugs, when these are fixed we should use that for all bindings
Expand Down

0 comments on commit 4194d11

Please sign in to comment.