On Wed, 8 Nov 2023 21:08:03 GMT, Viktor Klang wrote:
>> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461)
>
> Viktor Klang has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - Addressing review feedback
> - Make
On Thu, 9 Nov 2023 09:42:41 GMT, Viktor Klang wrote:
>> src/java.base/share/classes/java/util/stream/Gatherers.java line 326:
>>
>>> 324:
>>> 325: /**
>>> 326: * Gathers elements into fixed-size windows. The last window may
>>> contain
>>
>> Returns a ...
>
> You mean "Returns a
On Mon, 30 Oct 2023 15:38:35 GMT, Viktor Klang wrote:
> This Pull-Request implements [JEP-461](https://openjdk.org/jeps/461)
src/java.base/share/classes/java/util/stream/Gatherer.java line 38:
> 36: /**
> 37: * An intermediate operation that transforms a stream of input elements
> into a
>
On Wed, 8 Nov 2023 16:03:22 GMT, Ryan Wallace wrote:
> Added and tested and it still works as expected.
@ryawalla Would it be possible to update the PR description so there's a clear
summary on the updated proposal? I think it aligns the behavior with what
happens when a bad file path is
On Tue, 7 Nov 2023 03:20:45 GMT, Alexey Semenyuk wrote:
> Remove `-Xmx512m` from the jtreg `@run` command as @AlanBateman suggested
Marked as reviewed by alanb (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/16535#pullrequestreview-1718248698
On Tue, 7 Nov 2023 15:45:02 GMT, Alan Bateman wrote:
>> Jim Laskey has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains 14 addi
On Tue, 7 Nov 2023 12:29:12 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Tue, 7 Nov 2023 15:45:02 GMT, Alan Bateman wrote:
>> Jim Laskey has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains 14 addi
On Tue, 7 Nov 2023 12:29:12 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Tue, 7 Nov 2023 11:11:31 GMT, Erik Gahlin wrote:
>> Could I have a review of a PR that removes the bytecode instrumentation for
>> the exception events.
>>
>> Testing: jdk/jdk/jfr + tier1 + tier2
>
> Erik Gahlin has updated the pull request incrementally with one additional
> commit since
On Tue, 7 Nov 2023 11:11:31 GMT, Erik Gahlin wrote:
>> Could I have a review of a PR that removes the bytecode instrumentation for
>> the exception events.
>>
>> Testing: jdk/jdk/jfr + tier1 + tier2
>
> Erik Gahlin has updated the pull request incrementally with one additional
> commit since
On Tue, 7 Nov 2023 02:10:29 GMT, Erik Gahlin wrote:
> I filed an issue to investigate if there is a problem with SOE, or if the OOM
> check is really needed now. https://bugs.openjdk.org/browse/JDK-8319579
>
> Regardless of outcome, It would be good to document the results of the
>
On Tue, 7 Nov 2023 02:10:29 GMT, Erik Gahlin wrote:
> I filed an issue to investigate if there is a problem with SOE, or if the OOM
> check is really needed now. https://bugs.openjdk.org/browse/JDK-8319579
>
> Regardless of outcome, It would be good to document the results of the
>
On Mon, 6 Nov 2023 22:52:50 GMT, Erik Gahlin wrote:
>> Could I have a review of a PR that removes the bytecode instrumentation for
>> the exception events.
>>
>> Testing: jdk/jdk/jfr + tier1 + tier2
>
> Erik Gahlin has updated the pull request incrementally with one additional
> commit since
On Mon, 6 Nov 2023 22:52:50 GMT, Erik Gahlin wrote:
>> Could I have a review of a PR that removes the bytecode instrumentation for
>> the exception events.
>>
>> Testing: jdk/jdk/jfr + tier1 + tier2
>
> Erik Gahlin has updated the pull request incrementally with one additional
> commit since
On Mon, 6 Nov 2023 20:22:43 GMT, Mandy Chung wrote:
>> This is a regression caused by JDK-8302791. IAE should be thrown when an
>> interface is not visible to the given class loader but NPE is thrown instead
>> when the loader is null. The boot loader has no name and so the fix will
>>
On Tue, 7 Nov 2023 03:20:45 GMT, Alexey Semenyuk wrote:
> Remove `-Xmx512m` from the jtreg `@run` command as @AlanBateman suggested
Can you confirm that debug builds are passing with -XX:+UseZGC now?
-
PR Comment: https://git.openjdk.org/jdk/pull/16535#issuecomment-1797889256
On Mon, 6 Nov 2023 19:12:28 GMT, Mandy Chung wrote:
> This is a regression caused by JDK-8302791. IAE should be thrown when an
> interface is not visible to the given class loader but NPE is thrown instead
> when the loader is null. The boot loader has no name and so the fix will
> print
On Fri, 3 Nov 2023 12:19:07 GMT, Erik Gahlin wrote:
> Could I have a review of a PR that removes the bytecode instrumentation for
> the exception events.
>
> Testing: jdk/jdk/jfr + tier1 + tier2
src/java.base/share/classes/jdk/internal/event/ThrowableTracer.java line 37:
> 35: private
On Fri, 3 Nov 2023 12:19:07 GMT, Erik Gahlin wrote:
> Could I have a review of a PR that removes the bytecode instrumentation for
> the exception events.
>
> Testing: jdk/jdk/jfr + tier1 + tier2
src/java.base/share/classes/jdk/internal/event/ThrowableTracer.java line 37:
> 35: private
On Mon, 30 Oct 2023 16:16:52 GMT, Ryan Wallace wrote:
> Hi all,
>
> Please review this fix for jar tool not producing archive if there is a
> missing file supplied. Fix is to throw an exception and exit processing when
> a missing file is supplied. Current behaviour will recognise missing
On Mon, 30 Oct 2023 16:16:52 GMT, Ryan Wallace wrote:
> Hi all,
>
> Please review this fix for jar tool not producing archive if there is a
> missing file supplied. Fix is to throw an exception and exit processing when
> a missing file is supplied. Current behaviour will recognise missing
On Mon, 6 Nov 2023 05:34:04 GMT, David Holmes wrote:
>> test/hotspot/jtreg/serviceability/jvmti/stress/StackTrace/Suspended/libGetStackTraceSuspendedStress.cpp
>> line 125:
>>
>>> 123: err = jvmti->SetEventNotificationMode(JVMTI_ENABLE,
>>> JVMTI_EVENT_SINGLE_STEP, vthread);
>>> 124: if
On Fri, 3 Nov 2023 15:29:25 GMT, Jim Laskey wrote:
>> Update String Templates for a second preview. With the addition of
>>
>> - Expression type and throws are determined from the `process` method of the
>> processor type and not the processor type.
>>
>> - Qualified `STR` and `RAW` are
On Mon, 16 Oct 2023 14:31:46 GMT, Jim Laskey wrote:
> Wasn't sure about that. Thx.
When in doubt, JEP 12. Alex provided good guidance for API authors on how
`@since` should be used with preview APIs.
-
PR Review Comment:
On Mon, 16 Oct 2023 14:31:46 GMT, Jim Laskey wrote:
> Wasn't sure about that. Thx.
When in doubt, JEP 12. Alex provided good guidance for API authors on how
`@since` should be used with preview APIs.
-
PR Review Comment:
On Fri, 3 Nov 2023 15:29:25 GMT, Jim Laskey wrote:
>> Update String Templates for a second preview. With the addition of
>>
>> - Expression type and throws are determined from the `process` method of the
>> processor type and not the processor type.
>>
>> - Qualified `STR` and `RAW` are
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote:
> The various Zip/Jar-file related Java APIs have some long-standing
> differences or peculiarities with respect to the ZIP-file specification or
> compared to other implementations which should be documented in the API-doc.
> This
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote:
> The various Zip/Jar-file related Java APIs have some long-standing
> differences or peculiarities with respect to the ZIP-file specification or
> compared to other implementations which should be documented in the API-doc.
> This
On Mon, 30 Oct 2023 17:26:53 GMT, Yakov Shafranovich wrote:
> The various Zip/Jar-file related Java APIs have some long-standing
> differences or peculiarities with respect to the ZIP-file specification or
> compared to other implementations which should be documented in the API-doc.
> This
On Fri, 3 Nov 2023 05:43:38 GMT, David Holmes wrote:
> I don't understand what this is trying to do. If any virtual thread has an
> uncaught exception then other virtual threads will throw it wrapped in a
> RuntimeException. But the first virtual thread that completes (normally or by
>
On Thu, 2 Nov 2023 22:20:02 GMT, Martin Balao wrote:
>> src/java.base/share/classes/java/security/Security.java line 243:
>>
>>> 241: if (connection instanceof FileURLConnection
>>> fileConnection) {
>>> 242: // A local file URL can be interpreted as a Path
>>> 243:
On Thu, 2 Nov 2023 22:20:02 GMT, Martin Balao wrote:
>> src/java.base/share/classes/java/security/Security.java line 243:
>>
>>> 241: if (connection instanceof FileURLConnection
>>> fileConnection) {
>>> 242: // A local file URL can be interpreted as a Path
>>> 243:
On Tue, 31 Oct 2023 16:11:47 GMT, Joe Darcy wrote:
>> Clarify the intention of tier 1 tests. I'll reflow the paragraph and
>> regenerate the HTML file once the wording is agreed upon.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
On Thu, 2 Nov 2023 19:07:48 GMT, Francisco Ferrari Bihurriet
wrote:
> The implementation of this proposal is based on the requirements,
> specification and design choices described in the [JDK-8319332] ticket and
> its respective CSR [JDK-8319333]. What follows are implementation notes
>
On Thu, 2 Nov 2023 19:07:48 GMT, Francisco Ferrari Bihurriet
wrote:
> The implementation of this proposal is based on the requirements,
> specification and design choices described in the [JDK-8319332] ticket and
> its respective CSR [JDK-8319333]. What follows are implementation notes
>
On Thu, 2 Nov 2023 17:53:12 GMT, Jim Laskey wrote:
> I thought it was a clearer way to get rid of the side effect.
It means LauncherHelper.checkAndLoadMain description will need to be updated as
it says that it checks the public static void main method. But okay, I'll go
through what you have
On Thu, 2 Nov 2023 17:53:12 GMT, Jim Laskey wrote:
> I thought it was a clearer way to get rid of the side effect.
It means LauncherHelper.checkAndLoadMain description will need to be updated as
it says that it checks the public static void main method. But okay, I'll go
through what you have
On Thu, 2 Nov 2023 15:45:03 GMT, Jim Laskey wrote:
> Amazingly enough I seem to have the JNI straight on the first try.
Oh, I wasn't expecting that. My comment was suggesting we change
LauncherHelper.checkAndLoadMain to set both appClass and mainType after the
validate succeeds.
On Thu, 2 Nov 2023 15:45:03 GMT, Jim Laskey wrote:
> Amazingly enough I seem to have the JNI straight on the first try.
Oh, I wasn't expecting that. My comment was suggesting we change
LauncherHelper.checkAndLoadMain to set both appClass and mainType after the
validate succeeds.
On Thu, 2 Nov 2023 16:19:35 GMT, Mandy Chung wrote:
>> Tool modules can be created via `jmod --main-class` option such that
>> `ModuleMainClass` attribute will be added in `module-info.class` and the
>> module's main class can be launched via `java -m ` without
>> specifying the name of the
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Thu, 2 Nov 2023 12:33:27 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Thu, 2 Nov 2023 10:57:15 GMT, Hannes Wallnöfer wrote:
>> This change fixes two errors in inline HTML styles in the `java.lang`
>> package:
>>
>> - wrong CSS property name in `java.lang.String`
>> - CSS declaration terminated by colon instead of semicolon in
>> `java.lang.Thread`
>>
>>
On Wed, 1 Nov 2023 19:14:22 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Wed, 1 Nov 2023 19:14:22 GMT, Jim Laskey wrote:
>> Address changes from JEP 445 to JEP 463.
>>
>> - Move from a SYNTHETIC unnamed class to a MANDATED implicit class.
>>
>> - Don't mark class on read.
>>
>> - Remove reflection and annotation processing related to unnamed classes.
>>
>> -
On Wed, 1 Nov 2023 19:58:07 GMT, Mandy Chung wrote:
> Tool modules can be created via `jmod --main-class` option such that
> `ModuleMainClass` attribute will be added in `module-info.class` and the
> module's main class can be launched via `java -m ` without
> specifying the name of the main
On Tue, 31 Oct 2023 20:45:11 GMT, Calvin Cheung wrote:
> Do you prefer the `canArchive` setting be inside `if
> (CDS.isDumpingStaticArchive())` like the following?
>
> ```
> if (CDS.isDumpingStaticArchive())
> canArchive = true;
> ```
That's fine. It has been harmless to have
On Wed, 1 Nov 2023 04:34:35 GMT, Calvin Cheung wrote:
>> Please review this changeset for adding support for `--module` (-m) option
>> for CDS.
>> Changes in the `ModuleBootstrap.java` are needed so that the
>> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if
>> the
On Tue, 31 Oct 2023 20:14:45 GMT, Joe Darcy wrote:
> Due to their longer-than-typical running time and because the vector API is
> not (yet) in the base module, move the vector API tests in tier 1 to tier 2.
jdk_vector_sanity was added by JDK-8295970 to get some coverage in tier1. That
On Tue, 10 Oct 2023 15:09:41 GMT, Adam Sotona wrote:
> JImageValidator has been recently converted to use ClassFile API and original
> weak validation of class files was replaced with full class verification.
> Unfortunately full class verification is too strong as it includes class
>
On Mon, 16 Oct 2023 16:31:04 GMT, Brian Burkhalter wrote:
> The CSR was created.
Thanks. I changed the "Compatibility Risk" to medium and expanded its
description to more clearly explain the compatibility risk to fix this long
standing issue.
-
PR Comment:
On Mon, 30 Oct 2023 15:07:49 GMT, Viktor Klang wrote:
>> Discussed with @DougLea and adjusting the queue capacity to at least the
>> number of participating threads seems like the most sensible fix.
>
> Viktor Klang has updated the pull request incrementally with one additional
> commit since
On Tue, 31 Oct 2023 06:17:34 GMT, Calvin Cheung wrote:
> I've added the following field in `ArchivedModuleGraph` so that the
> `get(String mainModuleName)` will check the `mainModule` before returning
> `archivedModuleGraph`. `private static String mainModule;` The `mainModule`
> field is
On Tue, 31 Oct 2023 06:11:50 GMT, Calvin Cheung wrote:
>> Please review this changeset for adding support for `--module` (-m) option
>> for CDS.
>> Changes in the `ModuleBootstrap.java` are needed so that the
>> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if
>>
On Mon, 30 Oct 2023 17:16:54 GMT, Andrew Haley wrote:
>> The bug here is a thinko in `ScopedValue.scopedValueBindings()`.
>>
>> If the JVM runs out of resources, we throw a `VirtualMachineError`. Running
>> out of resources can happen at almost any time, and can happen while
>>
On Tue, 31 Oct 2023 00:23:44 GMT, Joe Darcy wrote:
>> Clarify the intention of tier 1 tests. I'll reflow the paragraph and
>> regenerate the HTML file once the wording is agreed upon.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
On Fri, 1 Sep 2023 08:29:41 GMT, Alan Bateman wrote:
>> I think you may have missed the comment in the JBS issue. Logging means
>> running potentially arbitrary code, doing this at Runtime.halt time is
>> problematic. I thought the conclusion from the work on Runtime.exit
On Mon, 30 Oct 2023 15:57:23 GMT, Andrew Haley wrote:
> The bug here is a thinko in `ScopedValue.scopedValueBindings()`.
>
> If the JVM runs out of resources, we throw a `VirtualMachineError`. Running
> out of resources can happen at almost any time, and can happen while
> `ScopedValue`'s
On Sat, 28 Oct 2023 21:03:50 GMT, Joe Darcy wrote:
> So in terms of a sentence or two of guidance, I think "aim for 10 seconds or
> less almost all of the time for a tier 1 test" is reasonable in this context.
Yes, I think making it an aspiration would be better.
In passing, you have
On Fri, 27 Oct 2023 19:03:36 GMT, Calvin Cheung wrote:
> The `ArchivedModuleGraph.java` wasn't changed. So if `-m` is not specified,
> the `archivedModuleGraph` is non-null; if `-m` is specified, the
> `archivedModuleGraph` is null. So running `java -version`, the
> archivedModuleGraph is
On Fri, 27 Oct 2023 16:23:04 GMT, Calvin Cheung wrote:
> I reran the script you sent me few days ago and got the expected results with
> the latest changes. The checking of the main module name matches between dump
> time and runtime is performed in the VM code. If an archive (even the default
On Thu, 26 Oct 2023 21:08:07 GMT, Calvin Cheung wrote:
> I've pushed another update with the following changes:
>
> * in the VM code, skip archiving full module graph if there's an incubator
> module by checking if the ArchivedBootLayer::archivedBootLayer is available;
I think we still have
On Sat, 22 Jul 2023 04:44:21 GMT, Stuart Marks wrote:
>> See https://bugs.java.com/bugdatabase/view_bug?bug_id=8305734 and
>> https://bugs.java.com/bugdatabase/view_bug?bug_id=JDK-8311905
>
> Hi, thanks for the background and the discussion of the alternatives. I'm not
> sure that drafting the
On Thu, 26 Oct 2023 19:38:55 GMT, Joe Darcy wrote:
>> Clarify the intention of tier 1 tests. I'll reflow the paragraph and
>> regenerate the HTML file once the wording is agreed upon.
>
> Joe Darcy has updated the pull request incrementally with one additional
> commit since the last revision:
On Tue, 26 Sep 2023 14:57:58 GMT, Alan Bateman wrote:
>> Fixes the issue (hopefully) by resolving automatic modules and automatic
>> module dependencies after propagation of non-automatic transitive
>> dependencies. The module tests run.
>>
>> I also a
On Wed, 25 Oct 2023 11:16:54 GMT, Jorn Vernee wrote:
>> The result of `FindClass` is a local JNI handle (in
>> `find_class_from_class_loader`, called from `jni_FindClass` [1]). As such,
>> we need to wrap the return value of `FindClass` in a global reference when
>> storing it inside
On Wed, 25 Oct 2023 21:08:01 GMT, Leonid Mesnik wrote:
> The jtreg starts the main thread in a separate ThreadGroup and checks
> unhandled exceptions for this group. However, it doesn't catch all unhandled
> exceptions. There is a jtreg issue for this
>
On Tue, 24 Oct 2023 17:01:35 GMT, Jorn Vernee wrote:
> The result of `FindClass` is a local JNI handle (in
> `find_class_from_class_loader`, called from `jni_FindClass` [1]). As such, we
> need to wrap the return value of `FindClass` in a global reference when
> storing it inside
On Mon, 23 Oct 2023 15:28:31 GMT, Viktor Klang wrote:
> Resolves a visibility issue in SubmissionPublisher by making the `waiter`
> member of BufferedSubscription *volatile*.
Marked as reviewed by alanb (Reviewer).
-
PR Review:
On Mon, 23 Oct 2023 18:26:51 GMT, Raffaello Giulietti
wrote:
>> Please review this simple fix to restore the original exception message that
>> existed before [16050](https://github.com/openjdk/jdk/pull/16050).
>
> Raffaello Giulietti has updated the pull request incrementally with one
>
On Sat, 21 Oct 2023 10:56:01 GMT, Doug Simon wrote:
>> The Graal code base has
>> [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33)
>> its module to `jdk.compiler.graal` as part of preparations for Project
>> Galahad. Due to the way Java modules work,
On Sat, 21 Oct 2023 10:56:01 GMT, Doug Simon wrote:
>> The Graal code base has
>> [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33)
>> its module to `jdk.compiler.graal` as part of preparations for Project
>> Galahad. Due to the way Java modules work,
On Mon, 23 Oct 2023 09:14:06 GMT, Sean Coffey wrote:
>> Fix up java.util.zip.ZipFile$Source hashCode() impl so that duplicate Source
>> objects aren't created for the same zip file.
>
> Sean Coffey has updated the pull request incrementally with one additional
> commit since the last revision:
On Sat, 21 Oct 2023 00:56:21 GMT, Joe Darcy wrote:
> Add informative notes to BigInteger and BigDecimal about possible running
> times, etc.
The wording looks okay to me and I expect it will appear as an API note in
BigInteger.
However, for BigDecimal I suspect the h2 headings (for the next
On Fri, 20 Oct 2023 20:24:26 GMT, Calvin Cheung wrote:
> I tried the above but got the following build error:
>
> ```
> Optimizing the exploded image
> Error occurred during initialization of boot layer
> java.lang.NullPointerException
> ExplodedImageOptimize.gmk:39: recipe for target
>
On Sat, 21 Oct 2023 07:42:06 GMT, Alan Bateman wrote:
>>> Yes, because of the following code further up in the same method:
>>
>> I think what you are actually doing here is supporting archiving of the boot
>> layer when the main module transitively depends on an inc
On Sat, 21 Oct 2023 06:54:16 GMT, Alan Bateman wrote:
> > Yes, because of the following code further up in the same method:
>
> I think what you are actually doing here is supporting archiving of the boot
> layer when the main module transitively depends on an incubator modu
On Fri, 20 Oct 2023 20:22:31 GMT, Calvin Cheung wrote:
> Yes, because of the following code further up in the same method:
I think what you are actually doing here is supporting archiving of the boot
layer when the main module transitively depends on an incubator module. We
might have to add
On Wed, 18 Oct 2023 15:59:13 GMT, Sean Coffey wrote:
>> Fix up java.util.zip.ZipFile$Source hashCode() impl so that duplicate Source
>> objects aren't created for the same zip file.
>
> Sean Coffey has updated the pull request incrementally with one additional
> commit since the last revision:
On Fri, 20 Oct 2023 15:45:50 GMT, Doug Simon wrote:
>> The Graal code base has
>> [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33)
>> its module to `jdk.compiler.graal` as part of preparations for Project
>> Galahad. Due to the way Java modules work,
On Fri, 20 Oct 2023 15:45:50 GMT, Doug Simon wrote:
>> The Graal code base has
>> [renamed](https://github.com/oracle/graal/commit/1e41203d10db321f86723eac90f6cd0573b08b33)
>> its module to `jdk.compiler.graal` as part of preparations for Project
>> Galahad. Due to the way Java modules work,
On Fri, 20 Oct 2023 13:35:59 GMT, Jorn Vernee wrote:
> Port over stylistic changes from the panama-foreign repo. The following PRs
> are included:
>
> - https://github.com/openjdk/panama-foreign/pull/863
> - https://github.com/openjdk/panama-foreign/pull/888
> -
On Thu, 19 Oct 2023 15:21:55 GMT, Calvin Cheung wrote:
>> Please review this changeset for adding support for `--module` (-m) option
>> for CDS.
>> Changes in the `ModuleBootstrap.java` are needed so that the
>> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if
>>
On Thu, 19 Oct 2023 15:21:55 GMT, Calvin Cheung wrote:
>> Please review this changeset for adding support for `--module` (-m) option
>> for CDS.
>> Changes in the `ModuleBootstrap.java` are needed so that the
>> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if
>>
On Thu, 19 Oct 2023 15:21:55 GMT, Calvin Cheung wrote:
>> Please review this changeset for adding support for `--module` (-m) option
>> for CDS.
>> Changes in the `ModuleBootstrap.java` are needed so that the
>> `ArchivedModuleGraph.archive` and `ArchivedBootLayer.archive` are called if
>>
On Fri, 20 Oct 2023 13:59:16 GMT, Maurizio Cimadamore
wrote:
> This simple PR removes the extra verbiage from the javadoc of all the
> restricted methods.
> Since now javadoc will automatically generate better "bubbles" for restricted
> methods, this wording is no longer necessary.
Marked as
On Fri, 20 Oct 2023 08:03:01 GMT, Daniel Jeliński wrote:
> ProxySelector.getDefault may return null if someone explicitly calls
> ProxySelector.setDefault(null); otherwise it's initialized to an instance of
> DefaultProxySelector
Right, I guess part of my puzzlement is to understand why
On Thu, 19 Oct 2023 12:10:02 GMT, Daniel Jeliński wrote:
> This patch fixes the encoding of IPv6 addresses in the URI provided to
> ProxySelector.
>
> The original code percent-encoded the square brackets added around the IPv6
> addresses, producing URIs that could not be parsed correctly.
>
On Sat, 14 Oct 2023 18:07:45 GMT, Alan Bateman wrote:
> JVMTI is notified after a virtual thread unmounts. Right not, this
> notification is done after the virtual thread has finished parking or
> yielding so it's possible for the virtual to continue, and be mounted on a
> diffe
On 19/10/2023 14:12, Severin Gehwolf wrote:
Hi,
Could I please get some reviews on the CSR for a feature that would
allow runtime image-based jlinks (no need for the packaged modules -
jmods folder - being present)?
https://bugs.openjdk.org/browse/JDK-8317420
It would be particularly helpful
On Sat, 14 Oct 2023 18:07:45 GMT, Alan Bateman wrote:
> JVMTI is notified after a virtual thread unmounts. Right not, this
> notification is done after the virtual thread has finished parking or
> yielding so it's possible for the virtual to continue, and be mounted on a
> diffe
hile in the area, I've removed @ChangesCurrentThread from VirtualThread.run.
> This annotation was in place to workaround an issue with the notifyJvmtiXXX
> instrinsics, fixed recently by JDK-8316130.
>
> Testing: tier1-6.
Alan Bateman has updated the pull request with a n
On Thu, 19 Oct 2023 07:04:50 GMT, Chen Liang wrote:
> NoRepl means "no replication", which is not what these APIs are for.
I assume it's "no replace", meaning it reports the error rather than use the
coder's replacement value.
I agree it could be renamed but I think look at CodingErrorAction
On Thu, 19 Oct 2023 07:04:50 GMT, Chen Liang wrote:
> NoRepl means "no replication", which is not what these APIs are for.
I assume it's "no replace", meaning it reports the error rather than use the
coder's replacement value.
I agree it could be renamed but I think look at CodingErrorAction
On Wed, 18 Oct 2023 23:37:58 GMT, Serguei Spitsyn wrote:
> Looks good. It is more elegant now. I was thinking if it'd make sense to move:
>
> * `notifyJvmtiMount(/*hide*/true);` at start of `mount()`
> * `notifyJvmtiUnmount(/*hide*/false);` at end of unmount()
> but I guess you wanted to have
On Wed, 18 Oct 2023 16:27:58 GMT, Roger Riggs wrote:
> The new usages that are driving this change would be better served by a
> method dedicated to creating latin1 strings from a caller provided byte
> array. The JavaLangAccess shared secret interface is already overused, but it
> is
On Wed, 18 Oct 2023 16:27:58 GMT, Roger Riggs wrote:
> The new usages that are driving this change would be better served by a
> method dedicated to creating latin1 strings from a caller provided byte
> array. The JavaLangAccess shared secret interface is already overused, but it
> is
801 - 900 of 18095 matches
Mail list logo
