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

Calling method on integer literals (like: 5.ToString() ) #20

Closed
nborelli opened this issue Apr 30, 2014 · 1 comment
Closed

Calling method on integer literals (like: 5.ToString() ) #20

nborelli opened this issue Apr 30, 2014 · 1 comment
Labels
Milestone

Comments

@nborelli
Copy link

Hi Davide,

It is not much of an issue at all, but I found a minor defect in number parsing. It seems that number parsing is a little too eager.

This will not work:
5.ToString()

But 5 .ToString() or (5).ToString() or 5.0.ToString() will.

As I said earlier, a very minor problem and easy to work around. Would also probably not be encountered very often.

DynamicExpresso.ParseException: Digit expected (at index 2).
at DynamicExpresso.ExpressionParser.ValidateDigit()
at DynamicExpresso.ExpressionParser.NextToken()
at DynamicExpresso.ExpressionParser..ctor(String expression, Type expressionType, ParameterExpression[] parameters, ParserSettings settings)
at DynamicExpresso.Interpreter.ParseExpression(String expressionText, Type expressionType, ParameterExpression[] parameters)
at DynamicExpresso.Interpreter.Parse(String expressionText, Type expressionType, Parameter[] parameters)
at DynamicExpresso.Interpreter.Parse(String expressionText, Parameter[] parameters)
at DynamicExpressoDemo.MainViewModel.Evaluate() in c:\Users\neal.borelli\Documents\Visual Studio 2013\Projects\DynamicExpressoDemo\DynamicExpressoDemo\MainViewModel.cs:line 128

@davideicardi davideicardi added this to the 0.11.2 milestone May 3, 2014
@davideicardi davideicardi changed the title Minor defect in number parsing. Calling method on integer literals (like: 5.ToString() ) May 3, 2014
@davideicardi
Copy link
Member

Just fixed in version 0.11.2.0. Thank you for bug report!

samuelcadieux pushed a commit to samuelcadieux/DynamicExpresso that referenced this issue May 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants