[jdk8u-dev] RFR: 8074840: Resolve disabled warnings for libjli and libjli_static
Paul Hohensee
phh at openjdk.org
Thu Dec 11 22:33:14 UTC 2025
On Mon, 8 Dec 2025 18:55:16 GMT, Zdenek Zambersky <zzambers at openjdk.org> wrote:
> Backport, which fixes compiler errors/warnings in libjli.
>
> **Motivation:**
> - to fix compilation error (and some warnings) on macos-15 (clang):
>
> ...
> 2025-11-19T17:50:19.3284070Z /Users/runner/work/jdk8u-dev/jdk8u-dev/jdk/jdk/src/share/bin/parse_manifest.c:680:12: note: use '==' to turn this assignment into an equality comparison
> 2025-11-19T17:50:19.3285850Z 680 | if (rc = find_file(fd, &entry, manifest_name) != 0) {
> 2025-11-19T17:50:19.3286370Z | ^
> 2025-11-19T17:50:19.3287050Z | ==
> 2025-11-19T17:50:19.3287530Z 4 warnings generated.
> 2025-11-19T17:50:19.3289930Z /Users/runner/work/jdk8u-dev/jdk8u-dev/jdk/jdk/src/share/bin/splashscreen_stubs.c:64:30: error: incompatible pointer to integer conversion returning 'void *' from a function with result type 'int' [-Wint-conversion]
> 2025-11-19T17:50:19.3291940Z 64 | INVOKE(SplashLoadMemory, NULL)(pdata, size);
> 2025-11-19T17:50:19.3292900Z | ^~~~
> ...
>
>
> **Differences in 8:**
> - excluded changes in `jdk/make/lib/CoreLibraries.gmk` due to differences in build system (not applicable to 8)
> - excluded cast of `STACK_SIZE_MINIMUM` in `java.c`, as [JDK-6762191](https://bugs.openjdk.org/browse/JDK-6762191), which adds that code, is missing in 8
> - excluded modifications to `readAt` function in `parse_manifest.c`, as function was added by [JDK-8073158](https://bugs.openjdk.org/browse/JDK-8073158), which is missing in 8
> - copyright differences due to missing backports or backport applied out of order (some had to be updated manually)
> - differences in paths
>
> Apart from above differences most of remaining code changes applied cleanly (with updated paths).
>
> **Testing:**
> Tier1 in GHA: **OK** (Failures of `TestCPUAwareness.java` are [unrelated](https://github.com/openjdk/jdk8u-dev/pull/727))
Marked as reviewed by phh (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk8u-dev/pull/733#pullrequestreview-3569579870
More information about the jdk8u-dev
mailing list