RFR: 8281275: Upgrading from 8 to 11 no longer accepts '/' as filepath separator in gc paths
David Holmes
dholmes at openjdk.java.net
Thu Feb 10 04:59:19 UTC 2022
Please review this trivial fix for UL parsing logic on Windows. We already special-cased paths like c:\logfile but didn't allow for c:/logfile. As noted in the bug report this is a regression from using -Xloggc where the c:/ path was allowed (and it even worked when Xloggc was converted internally to use UL - because it skipped the parsing logic and just directly set the file output!).
Testing:
- new gtest added by expanding existing parsing gtest
- tiers 1-3 Windows
Verified that new gtest fails without the fix.
Thanks,
David
-------------
Commit messages:
- Fix style issue
- 8281275: Upgrading from 8 to 11 no longer accepts '/' as filepath separator in gc paths
Changes: https://git.openjdk.java.net/jdk/pull/7415/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7415&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8281275
Stats: 28 lines in 2 files changed: 23 ins; 0 del; 5 mod
Patch: https://git.openjdk.java.net/jdk/pull/7415.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7415/head:pull/7415
PR: https://git.openjdk.java.net/jdk/pull/7415
More information about the hotspot-runtime-dev
mailing list