On Mon, 2 Aug 2021 10:52:26 GMT, Claes Redestad <redes...@openjdk.org> wrote:
>> Сергей Цыпанов has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains three commits: >> >> - Merge branch 'master' into 8270160 >> - Merge branch 'master' into 8270160 >> >> # Conflicts: >> # src/java.base/share/classes/java/lang/StringLatin1.java >> - 8270160: Remove redundant bounds check from AbstractStringBuilder.charAt() > > src/java.base/share/classes/java/lang/AbstractStringBuilder.java line 227: > >> 225: private void ensureCapacityInternal(int minimumCapacity) { >> 226: // overflow-conscious code >> 227: int oldCapacity = capacity(); > > This doesn't seem related to the intent of the patch, and might distort > startup profiles a bit. Reverted > src/java.base/share/classes/java/lang/StringLatin1.java line 47: > >> 45: public static char charAt(byte[] value, int index) { >> 46: checkIndex(index, value.length); >> 47: return getChar(value, index); > > Seems unrelated to the gist of the patch, and again might skew startup > profiles. Reverted ------------- PR: https://git.openjdk.java.net/jdk/pull/4738