RFR: 8303548: Arguments::get_default_shared_archive_path() should cache the result for future use

Matias Saavedra Silva matsaave at openjdk.org
Fri Mar 10 14:41:12 UTC 2023


On Fri, 10 Mar 2023 02:54:43 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> The method `Arguments::get_default_shared_archive_path()` generates a string containing a path every time it is called. Since this method is idempotent, the string only needs to be constructed once in the lifetime of the VM. This change caches the result of `Arguments::get_default_shared_archive_path()` and ensures the path string is only generated once.
>
> LGTM2
> 
> Thanks

Thank you @dholmes-ora  and @calvinccheung!

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

PR: https://git.openjdk.org/jdk/pull/12963


More information about the hotspot-runtime-dev mailing list