Jasper De Sutter
Jasper De Sutter
Yeah for testing this was all just some quick and dirty work, so I'd rather not include it in the repository. If you really need this you can add the...
Before these changes, Cow::Borrowed was returned 14 out of 174 times in fluent-bundle test cases. After the changes, I'm seeing 67 out of 174.
Great to see this project more alive again! I'll pick up my PRs that are still open, though it has been a while so it might take some time to...
Thanks for checking the perf impact, I do run the benchmarks but they don't seem too reliable on my laptop. I'm guessing the system load/cpu scaling has more impact than...
I've optimized the implementation a bit to work away a bounds check from char boundary checking. It does change the approach quite a bit, by changing the string slice instead...
Last try on my method! Shouldn't be possible to optimize much further.. I'm not able to spot the differences in the code you posted with my second commit, is it...
That explains why I wasn't seeing the same performance impact, I was comparing to main without that into_string commit included.. I've removed the optimization commit.
This could probably be solved by using https://docs.rs/fixed_decimal/0.5.5/fixed_decimal as the number representation, which supports arbitrary precision. Combined with the open PRs to switch to ICU4X for plurals and number formatting,...