-
Notifications
You must be signed in to change notification settings - Fork 60
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
conversion: add PrettyDec
, optimize Dec
, add benchmarks and fuzzing
#130
Conversation
Codecov Report
@@ Coverage Diff @@
## master #130 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 1543 1597 +54
=========================================
+ Hits 1543 1597 +54 |
The 'native' implementation is better on allocs, but way slower
|
Now it looks beter:
|
Now both of them are faster (and less alloc:y) than the |
|
PrettyDec
, optimize Dec
, add benchmarks and fuzzing
This PR implements the
Dec
method 'natively', and adds the API-methodPrettyDec
.It also adds benchmarks and a new fuzz-test for the changed methods.