RFR: 8202750: Reduce the use of get_canonical_path() in CDS [v2]

Calvin Cheung ccheung at openjdk.java.net
Wed Feb 17 16:45:54 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.

Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:

  per review comments from @iklam

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2581/files
  - new: https://git.openjdk.java.net/jdk/pull/2581/files/dc755372..bf940008

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2581&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2581&range=00-01

  Stats: 38 lines in 3 files changed: 17 ins; 14 del; 7 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