RFR: 8202750: Reduce the use of get_canonical_path() in CDS
Calvin Cheung
ccheung at openjdk.java.net
Tue Feb 16 02:10:45 UTC 2021
Please review this proposed change which includes:
- replace calls to `get_canonical_path()` with `ClassLoader::get_native_path()` which calls `os::native_path()`;
- factor out some code for opening a zip file into `ClassLoader::open_zip_file()`;
- modify `ClassLoader::get_canonical_path()` so that all buffer allocations are within the function.
Testing: Tiers 1 - 4.
-------------
Commit messages:
- Merge branch 'master' into 8202750-dir-canonical-form
- replace get_canonical_path() with get_native_path() in ClassLoader::record_result()
- fix the DirClasspathTest.java test failure on Windows
- simply the fix
- 8202750 (initial commit)
Changes: https://git.openjdk.java.net/jdk/pull/2581/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2581&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8202750
Stats: 60 lines in 2 files changed: 21 ins; 18 del; 21 mod
Patch: https://git.openjdk.java.net/jdk/pull/2581.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2581/head:pull/2581
PR: https://git.openjdk.java.net/jdk/pull/2581
More information about the hotspot-runtime-dev
mailing list