On Tue, 19 Apr 2022 08:50:09 GMT, Claes Redestad <redes...@openjdk.org> wrote:

> There is no pair of character values in the latin1 range (0-255) that will 
> make the condition at line 401 in `StringLatin1.java` true, so this test can 
> be removed.
> 
> Added a test and a microbenchmark (which as expected sees a few ns/op 
> improvement from this change). Took the opportunity to tune the default 
> settings for the micro to reduce runtime from 30+ minutes to 3 with no 
> discernible degradation of quality.

Nice Catch!

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

Marked as reviewed by rriggs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8292

Reply via email to