Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DynamicExpressionParser does not allow empty parameter lists. #79

Closed
groogiam opened this issue May 12, 2017 · 1 comment
Closed

DynamicExpressionParser does not allow empty parameter lists. #79

groogiam opened this issue May 12, 2017 · 1 comment
Assignees
Labels

Comments

@groogiam
Copy link

The DynamicExpressionParser does not allow empty parameters. The MS version allowed this so it seems like this library should allow this as well. Code snippet below reproduces the issue. Thanks.

var pEmpty = new ParameterExpression[] { };

var core = System.Linq.Dynamic.Core.DynamicExpressionParser.ParseLambda(pEmpty, null, "1+1").Compile(); //throws The collection argument 'parameters' must contain at least one element

var ms = System.Linq.Dynamic.DynamicExpression.ParseLambda(pEmpty, null, "1+1").Compile(); //this compiles and executes
@StefH StefH self-assigned this May 12, 2017
@StefH StefH added the bug label May 12, 2017
@StefH StefH changed the title System.Linq.Dynamic.Core.DynamicExpressionParser does not allow empty parameter lists. DynamicExpressionParser does not allow empty parameter lists. May 12, 2017
@StefH
Copy link
Collaborator

StefH commented May 12, 2017

1fb5134

@StefH StefH closed this as completed May 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants