Integrated: 8285001: Simplify StringLatin1.regionMatches

Claes Redestad redestad at openjdk.java.net
Tue Apr 19 20:33:08 UTC 2022


On Tue, 19 Apr 2022 08:50:09 GMT, Claes Redestad <redestad at 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.

This pull request has now been integrated.

Changeset: e307bc86
Author:    Claes Redestad <redestad at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/e307bc8694462568807021191f9653ee80a93ed1
Stats:     95 lines in 3 files changed: 88 ins; 3 del; 4 mod

8285001: Simplify StringLatin1.regionMatches

Reviewed-by: rriggs, naoto

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

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


More information about the core-libs-dev mailing list