Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v5]

2024-12-05 Thread Jaikiran Pai
On Thu, 5 Dec 2024 04:38:04 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v5]

2024-12-05 Thread Daniel Fuchs
On Thu, 5 Dec 2024 04:38:04 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v5]

2024-12-04 Thread Jaikiran Pai
On Thu, 5 Dec 2024 04:38:04 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v5]

2024-12-04 Thread Jaikiran Pai
> Can I please get a review of this change which proposes to address > https://bugs.openjdk.org/browse/JDK-8341551? > > The primary work in this PR is the specification of the previously existing > `sun.misc.URLClassPath.disableJarChecking` system property and how the > internal implementation

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v4]

2024-12-04 Thread Alan Bateman
On Wed, 4 Dec 2024 16:46:17 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Eirik Bjørsnøs
On Wed, 4 Dec 2024 16:46:14 GMT, Eirik Bjørsnøs wrote: >> We're stuck with the property name for compatibility, and the usage within >> the class is fairly limited. >> Generally, it is easier to understand the behavior having a feature that is >> enabled not a disable that is disabled. >> $0.02

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v4]

2024-12-04 Thread Roger Riggs
On Wed, 4 Dec 2024 16:46:17 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Eirik Bjørsnøs
On Wed, 4 Dec 2024 16:40:42 GMT, Roger Riggs wrote: >> Hello Eirik, I have received similar inputs in other places when discussing >> this change. I myself had to think a few times on which naming to follow >> here to make it easier to understand the code. I'll consider your and other >> input

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v3]

2024-12-04 Thread Jaikiran Pai
On Wed, 4 Dec 2024 16:35:09 GMT, Roger Riggs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Lance's suggestion - better comment on the method > > src/java.base/share/classes/jdk/internal/loader/URLClassPath.java l

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Roger Riggs
On Wed, 4 Dec 2024 16:35:19 GMT, Jaikiran Pai wrote: >> src/java.base/share/classes/jdk/internal/loader/URLClassPath.java line 89: >> >>> 87: // JAR check is disabled by default and will be enabled only if >>> the "disable JAR check" >>> 88: // system property has been set to "f

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v4]

2024-12-04 Thread Jaikiran Pai
> Can I please get a review of this change which proposes to address > https://bugs.openjdk.org/browse/JDK-8341551? > > The primary work in this PR is the specification of the previously existing > `sun.misc.URLClassPath.disableJarChecking` system property and how the > internal implementation

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v3]

2024-12-04 Thread Roger Riggs
On Wed, 4 Dec 2024 16:28:58 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Jaikiran Pai
On Wed, 4 Dec 2024 16:25:47 GMT, Eirik Bjørsnøs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> improve code comment > > src/java.base/share/classes/jdk/internal/loader/URLClassPath.java line 89: > >> 87:

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Jaikiran Pai
On Wed, 4 Dec 2024 16:12:39 GMT, Lance Andersen wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> improve code comment > > src/java.base/share/classes/jdk/internal/loader/URLClassPath.java line 657: > >> 655:

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Eirik Bjørsnøs
On Wed, 4 Dec 2024 15:46:28 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v3]

2024-12-04 Thread Jaikiran Pai
> Can I please get a review of this change which proposes to address > https://bugs.openjdk.org/browse/JDK-8341551? > > The primary work in this PR is the specification of the previously existing > `sun.misc.URLClassPath.disableJarChecking` system property and how the > internal implementation

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Daniel Fuchs
On Wed, 4 Dec 2024 15:58:41 GMT, Jaikiran Pai wrote: >> test/jdk/jdk/internal/loader/URLClassPath/JarCheckTest.java line 60: >> >>> 58: * @run junit/othervm -Dsun.misc.URLClassPath.disableJarChecking >>> JarCheckTest >>> 59: * @run junit/othervm -Dsun.misc.URLClassPath.disableJarChecking=true

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Lance Andersen
On Wed, 4 Dec 2024 15:46:28 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Jaikiran Pai
On Wed, 4 Dec 2024 14:53:19 GMT, Daniel Fuchs wrote: >> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> improve code comment > > test/jdk/jdk/internal/loader/URLClassPath/JarCheckTest.java line 60: > >> 58: * @run junit/ot

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Daniel Fuchs
On Wed, 4 Dec 2024 15:46:28 GMT, Jaikiran Pai wrote: >> Can I please get a review of this change which proposes to address >> https://bugs.openjdk.org/browse/JDK-8341551? >> >> The primary work in this PR is the specification of the previously existing >> `sun.misc.URLClassPath.disableJarChec

Re: RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486 [v2]

2024-12-04 Thread Jaikiran Pai
> Can I please get a review of this change which proposes to address > https://bugs.openjdk.org/browse/JDK-8341551? > > The primary work in this PR is the specification of the previously existing > `sun.misc.URLClassPath.disableJarChecking` system property and how the > internal implementation

RFR: 8341551: Revisit jdk.internal.loader.URLClassPath.JarLoader after JEP 486

2024-12-04 Thread Jaikiran Pai
Can I please get a review of this change which proposes to address https://bugs.openjdk.org/browse/JDK-8341551? The primary work in this PR is the specification of the previously existing `sun.misc.URLClassPath.disableJarChecking` system property and how the internal implementation of `java.ne