On Thu, 24 Aug 2023 10:05:10 GMT, Claes Redestad <[email protected]> wrote:

> While I lean towards being wary, I think there are definitely things in here 
> that is worth pursuing.
> 
> For example it seems profitable to avoid allocating and appending to a 
> `StringBuilder` for the case where we end up returning the original `String`, 
> so perhaps this would be better implemented using a fast-path loop:

Yes that's what I was trying to say in my first comment. I would be fine with 
keeping that part of the proposed changes.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/15354#issuecomment-1691419336

Reply via email to