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.

Sign up for GitHub

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

Jump to bottom

[Bug] Support for decimal qualifiers 'M' & 'm' #91

Closed
pferraris opened this issue Jun 25, 2017 · 1 comment
Closed

[Bug] Support for decimal qualifiers 'M' & 'm' #91

pferraris opened this issue Jun 25, 2017 · 1 comment
Assignees
@StefH
Labels
bug

Comments

@pferraris
Copy link
Contributor

I run into a problem when I tried to evaluate an expression with a decimal value (eg "0.5m").
ExpressionParser detects it as double, then PromoteExpression method try to convert it to decimal but keeps qualifier 'm' as part of text, and decimal.TryParse("0.11m", out e) returns 0.
@pferraris
Copy link
Contributor Author

I'm working to resolve this enhancement. I'll submit a pull request shortly.

@StefH StefH changed the title Support for decimal qualifiers 'M' & 'm' [Bug] Support for decimal qualifiers 'M' & 'm' Jun 25, 2017
@StefH StefH self-assigned this Jun 25, 2017
@StefH StefH added the bug label Jun 25, 2017
@StefH StefH closed this as completed in ab249d4 Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@StefH StefH

Labels
bug
Milestone
No milestone
Development

No branches or pull requests
2 participants
@StefH @pferraris