RFR: 8317951: Refactor loading of zip library to help resolve JDK-8315220 [v2]

Markus Grönlund mgronlun at openjdk.org
Thu Oct 19 08:31:54 UTC 2023


On Thu, 19 Oct 2023 02:39:32 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update for review comments.
>
> src/hotspot/share/utilities/zipLibrary.cpp line 152:
> 
>> 150: 
>> 151: jzentry* ZipLibrary::find_entry(jzfile* zip, const char* name, jint* sizeP, jint* nameLen) {
>> 152:   initialize();
> 
> Again don't we have to have initialized things to get the library we are now searching? The same goes for all(?) the other methods.

No, this is lazy library loading and initialisation as part of all the API entries (except close).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16149#discussion_r1365107412


More information about the hotspot-runtime-dev mailing list