XenoAmess
XenoAmess
@aherbert Hi. I changed the jmh codes according to your suggestions, and re-run performance test. full test at: https://pastebin.ubuntu.com/p/cqNKS35zSP/ in short: ``` Benchmark (data) (name) Mode Cnt Score Error Units...
Hi. I re-changed some codes and run a performance test. In short, ``` Benchmark (data) (name) Mode Cnt Score Error Units StringUtilsChompTest.singleString NULL old avgt 5 2.751 ? 0.091 ns/op...
@sebbASF > I don't think the RandomStrings test is a fair benchmark. Actually it is fair. See the string array generation function of that test.
@sebbASF Hi. > Sorry, I see now that the strings do have a mix of CR and LF endings (or neither). > However, the strings are all of length 2....
@sebbASF > I think the current huge list of input strings is more about testing functionality rather than performance. > > The method only cares about 3 characters: CR, LF...
Hi. I added some more lengths of Strings to the benchmark, as suggested by @sebbASF . Thanks. full benchmark at https://pastebin.ubuntu.com/p/yPH4xnKqZd/ In short, ``` Benchmark (data) (name) Mode Cnt Score...
@garydgregory rebased. please find some time to review. thanks.
also want to split StringUtils.startswith, endwith, prependIfMissing... now the call flow is like this: 1. user call startsWith(final CharSequence str, final CharSequence prefix) 2. give an additional boolean, and invoke...
Hi. According to `72 Hour Lazy Consensus` rule, if nobody against this pr, I will continue developing codes about this pr in 50 hours later, means spliting more functions in...
@garydgregory **Conclusion:** after doing the splition, it can speed up around 25%. ``` [INFO] --- exec-maven-plugin:1.6.0:exec (benchmark) @ commons-lang3 --- WARNING: An illegal reflective access operation has occurred WARNING: Illegal...