Skip to content

Commit

Permalink
Merge pull request #1768 from riganti/disable-expr-interpreter
Browse files Browse the repository at this point in the history
Disable Linq.Expression preferInterpretation for bindings
  • Loading branch information
exyi authored Jan 31, 2024
2 parents 9a4f1c1 + 4194d11 commit d976ed3
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 d976ed3

Please sign in to comment.