[jdk8u-dev] RFR: 8324184: Windows VS2010 build failed with "error C2275: 'int64_t'"
Andrew John Hughes
andrew at openjdk.org
Wed Jan 31 03:36:58 UTC 2024
Hi all,
This pull request contains a backport of commit [4d078930](https://github.com/openjdk/jdk21u/commit/4d078930eecfacb28a7c8324f233080eaf649334) from the [openjdk/jdk21u](https://git.openjdk.org/jdk21u) repository & the clean backport commit [4f80edf](https://github.com/openjdk/jdk17u-dev/commit/4f80edfae10e83f2709f297a553d2128712e4b51) from 17u, also now in 11u. A new bug ID, JDK-8324184, had to be created as JDK-8317331, the original bug, is private.
The commit being backported was authored by Coleen Phillimore on 2 Oct 2023 and had no reviewers.
The backport fixes the VS2010 build which is broken by the placing of the declarations of `low`, `high` and `index` in JDK-8314295 after statements; see https://github.com/gnu-andrew/jdk8u-dev/actions/runs/7549766959/job/20554342418. This positioning is disallowed by C90.
Adding `-Werror=declaration-after-statement` to the build flags replicated the failure on GCC 8. I will look at adding this to the build in a follow-up fix. It also needs a number of other changes to fix Linux-specific code which has never been built to this standard.
Backport was clean and built fine on GNU/Linux with GCC 8.
Thanks!
-------------
Commit messages:
- Backport 4f80edfae10e83f2709f297a553d2128712e4b51
Changes: https://git.openjdk.org/jdk8u-dev/pull/435/files
Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=435&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8324184
Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk8u-dev/pull/435.diff
Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/435/head:pull/435
PR: https://git.openjdk.org/jdk8u-dev/pull/435
More information about the jdk8u-dev
mailing list