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
max/min used with precision > 2 causes stack overflow #132
Comments
@GenuineRex 04/2021 |
The number of digits before the decimal point >= digits after the decimal point - 2 digits. For example, if there are 8 digits after the decimal point, there must be at least 6 digits before the decimal point. Not very suitable for recording Bitcoin transactions. Works:
Not working:
|
I opened a PR to fix that problem, hope it helps. I was facing the same problem and it fixed my issue. Its a problem related to the decimal precision number, it doesn't work well with numbers > 2 |
this line in the applyMask function limits the mask to using 2 decimal positions when max/min are also used.
The 100 assumes 2 decimal precision and min/max call applyMask again creating a stack overflow.
let newValue = integerValue + (decimalValue / 100);
The text was updated successfully, but these errors were encountered: