On Mon, 18 Sep 2023 22:39:13 GMT, Glavo <[email protected]> wrote: >> I mainly made these optimizations: >> >> * Avoid allocating `StringBuilder` when there are no characters in the URL >> that need to be encoded; >> * ~Implement a fast path for UTF-8.~ (Has been removed from this PR) >> >> In addition to improving performance, these optimizations also reduce >> temporary objects: >> >> * It no longer allocates any object when there are no characters in the URL >> that need to be encoded; >> * The initial size of StringBuilder is larger to avoid expansion as much as >> possible; >> * ~For UTF-8, the temporary `CharArrayWriter`, strings and byte arrays are >> no longer needed.~ (Has been removed from this PR) >> >> I also updated the tests to add more test cases. > > Glavo has updated the pull request incrementally with one additional commit > since the last revision: > > Update encodeByte
I ran the tier1 test with no failures. ------------- PR Comment: https://git.openjdk.org/jdk/pull/15354#issuecomment-1724592047
