Integrated: 8202750: Reduce the use of get_canonical_path() in CDS

Calvin Cheung ccheung at openjdk.java.net
Thu Feb 18 02:54:41 UTC 2021


On Tue, 16 Feb 2021 02:04:43 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: 5f308291
Author:    Calvin Cheung <ccheung at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/5f308291
Stats:     72 lines in 3 files changed: 27 ins; 21 del; 24 mod

8202750: Reduce the use of get_canonical_path() in CDS

Reviewed-by: minqi, iklam

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

PR: https://git.openjdk.java.net/jdk/pull/2581


More information about the hotspot-runtime-dev mailing list