> See bug report for reproducer and error message. The warning looks quite > superficial, and I initially suspected that it complains about `int len = > (int)strlen(str)` cast of `size_t`. But what is confusing is nearly the > similar code in `newStringCp1252` apparently does not produce the warning: > https://github.com/openjdk/jdk/blob/f5d1b5bda27c798347ae278cbf69725ed4be895c/src/java.base/share/native/libjava/jni_util.c#L533-L566 > > I propose we chalk it up to false/odd compiler warning and sync up > unproblematic `newStringCp1252` and `newString646_US` implementation to dodge > it. > > Additional testing: > - [x] Linux x86_32 build with GCC 11 > - [x] Linux x86_32 `tier1` test with GCC 9 > - [x] Linux x86_64 `tier1` test with GCC 9
Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision: - Also improve newSizedString8859_1 - Merge branch 'master' into JDK-8289049-gcc11-x86-32 - Init buf in newStringCp1252 - Another fix - Fix ------------- Changes: - all: https://git.openjdk.org/jdk/pull/9702/files - new: https://git.openjdk.org/jdk/pull/9702/files/adcbf91c..c2546de4 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=9702&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9702&range=01-02 Stats: 15250 lines in 640 files changed: 8464 ins; 4113 del; 2673 mod Patch: https://git.openjdk.org/jdk/pull/9702.diff Fetch: git fetch https://git.openjdk.org/jdk pull/9702/head:pull/9702 PR: https://git.openjdk.org/jdk/pull/9702
