Re: RFR: 8346239: Improve memory efficiency of JimageDiffGenerator

2024-12-29 Thread Matthias Baesken
On Thu, 19 Dec 2024 18:14:39 GMT, Severin Gehwolf wrote: > Please review this fairly simple change to improve how the > `JimageDiffGenerator` works. The original implementation is pretty naive and > read all bytes into memory and then compared them. This improved version only > reads bytes on

Integrated: 8346871: Improve robustness of java/util/zip/EntryCount64k.java test

2024-12-29 Thread Jaikiran Pai
On Sun, 29 Dec 2024 14:51:06 GMT, Jaikiran Pai wrote: > Can I please get a review of this test-only change which improves the > `test/jdk/java/util/zip/EntryCount64k.java` test code to allow for it to pass > when the JVM logs any warning when launching an application through `java > -jar ...`?

Re: RFR: 8346871: Improve robustness of java/util/zip/EntryCount64k.java test

2024-12-29 Thread Jaikiran Pai
On Sun, 29 Dec 2024 14:51:06 GMT, Jaikiran Pai wrote: > Can I please get a review of this test-only change which improves the > `test/jdk/java/util/zip/EntryCount64k.java` test code to allow for it to pass > when the JVM logs any warning when launching an application through `java > -jar ...`?

Re: RFR: 8346871: Improve robustness of java/util/zip/EntryCount64k.java test

2024-12-29 Thread Lance Andersen
On Sun, 29 Dec 2024 14:51:06 GMT, Jaikiran Pai wrote: > Can I please get a review of this test-only change which improves the > `test/jdk/java/util/zip/EntryCount64k.java` test code to allow for it to pass > when the JVM logs any warning when launching an application through `java > -jar ...`?

RFR: 8346871: Improve robustness of java/util/zip/EntryCount64k.java test

2024-12-29 Thread Jaikiran Pai
Can I please get a review of this test-only change which improves the `test/jdk/java/util/zip/EntryCount64k.java` test code to allow for it to pass when the JVM logs any warning when launching an application through `java -jar ...`? The proposed change merely relaxes a check to allow for additi

Integrated: 8302293: jar --create fails with IllegalArgumentException if archive name is shorter than 3 characters

2024-12-29 Thread Jaikiran Pai
On Fri, 20 Dec 2024 06:38:54 GMT, Jaikiran Pai wrote: > Can I please get a review for this change which proposes to fix the issue > noted in https://bugs.openjdk.org/browse/JDK-8302293? > > As noted in the issue, when creating a temporary file during the "jar > --create" operation, if the orig

Re: RFR: 8302293: jar --create fails with IllegalArgumentException if archive name is shorter than 3 characters [v2]

2024-12-29 Thread Jaikiran Pai
On Fri, 20 Dec 2024 14:15:14 GMT, Jaikiran Pai wrote: >> Can I please get a review for this change which proposes to fix the issue >> noted in https://bugs.openjdk.org/browse/JDK-8302293? >> >> As noted in the issue, when creating a temporary file during the "jar >> --create" operation, if the