On Tue, 14 Jun 2022 16:28:37 GMT, Paul Sandoz wrote:
> Update the code examples in the api notes of Long::compress/expand. Some
> constants need to be explicitly long values.
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.org/jdk19/pull/14
On Tue, 14 Jun 2022 12:18:52 GMT, Matthias Baesken wrote:
>> When trying to construct an LdapURL object with a bad input string (in this
>> example the _ in ad_jbs is causing issues), and not using
>> the backward compatibility flag -Dcom.sun.jndi.ldapURLParsing="legacy" we
>> run into the exce
On 14/06/2022 10:44, Andrey Turbanov wrote:
Hello.
During investigation of signal handling in JVM (for
https://github.com/openjdk/jdk/pull/9100#discussion_r894992558 )
I found out that sending USR2 crashes my JDK. (Linux fastdebug x64)
kill -USR2 1346792
# assert(thread != __null) failed: Missi
On Tue, 14 Jun 2022 11:36:36 GMT, Matthias Baesken wrote:
>> When trying to construct an LdapURL object with a bad input string (in this
>> example the _ in ad_jbs is causing issues), and not using
>> the backward compatibility flag -Dcom.sun.jndi.ldapURLParsing="legacy" we
>> run into the exce
JNI is updated in Java 19 so we need to define JNI_VERSION_19 and change
GetVersion to return this version.
test/hotspot/jtreg/native_sanity/JniVersion.java is updated to check that
JNI_VERSION_19 is returned. The native library in the JMX agent, and several
tests, define JNI_OnLoad that return
On Fri, 10 Jun 2022 13:23:51 GMT, David Holmes wrote:
> Expired Flags in 20:
>
> - FilterSpuriousWakeups
> - MinInliningThreshold
> - PrefetchFieldsAhead
>
> No remaining usages in code or tests.
>
> - UseHeavyMonitors (expired in PRODUCT ONLY)
>
> As this flag now only exists for debug buil
On Fri, 10 Jun 2022 18:19:42 GMT, Thiago Henrique Hüpner
wrote:
>> test/jdk/tools/jar/modularJar/Basic.java line 44:
>>
>>> 42:
>>> 43: import jdk.internal.module.ModuleReferenceImpl;
>>> 44: import jdk.internal.module.ModuleResolution;
>>
>> At some point we need to put in test infrastructur
On Fri, 10 Jun 2022 18:25:09 GMT, Thiago Henrique Hüpner
wrote:
>> 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved
>> flags is used
>
> Thiago Henrique Hüpner has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Ren
On Fri, 10 Jun 2022 16:30:57 GMT, Thiago Henrique Hüpner
wrote:
>> 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved
>> flags is used
>
> Thiago Henrique Hüpner has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Fix
On Fri, 10 Jun 2022 16:15:36 GMT, Joe Darcy wrote:
> There are many instanceof checks in the sun.reflection.annotation code; these
> would be improved by using pattern matching for instanceof.
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.org/jdk/pull/9129
On Fri, 10 Jun 2022 13:41:48 GMT, Matthias Baesken wrote:
> Hi Alan , sure we could use something like the already existing hostInfo of
> property jdk.includeInException private static final boolean
> enhancedExceptionText = SecurityProperties.includedInExceptions("hostInfo");
> and make the e
On Fri, 10 Jun 2022 12:16:17 GMT, Matthias Baesken wrote:
> When trying to construct an LdapURL object with a bad input string (in this
> example the _ in ad_jbs is causing issues), and not using
> the backward compatibility flag -Dcom.sun.jndi.ldapURLParsing="legacy" we run
> into the exceptio
On Wed, 8 Jun 2022 18:24:35 GMT, Paul Sandoz wrote:
>> Allow JDK modules that use preview features (preview language features or
>> preview API features from dependent modules) to participate without the need
>> to compile with `--enable-preview`.
>>
>> It's difficult to enable participation u
On Wed, 8 Jun 2022 15:46:24 GMT, Paul Sandoz wrote:
> Allow JDK modules that use preview features (preview language features or
> preview API features from dependent modules) to participate without the need
> to compile with `--enable-preview`.
>
> It's difficult to enable participation using
On Wed, 8 Jun 2022 14:51:59 GMT, Thiago Henrique Hüpner
wrote:
>> 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved
>> flags is used
>
> Thiago Henrique Hüpner has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Fix
Vote: yes
On Tue, 7 Jun 2022 00:45:17 GMT, Thiago Henrique Hüpner
wrote:
> 8287760: --do-not-resolve-by-default gets overwritten if --warn-if-resolved
> flags is used
The change looks okay but I think we should add a test. You'll have to check
the test tree to see if there are existing tests for incuba
Vote: yes
On Mon, 6 Jun 2022 16:54:29 GMT, Joe Darcy wrote:
>> src/java.base/share/classes/java/lang/Double.java line 683:
>>
>>> 681: * "[\\x00-\\x20]*");// Optional trailing "whitespace"
>>> 682: *
>>> 683: * if (Pattern.matches(fpRegex, myString)) // @link
>>> substring="Pattern.
On Mon, 6 Jun 2022 20:37:07 GMT, Joe Darcy wrote:
>> Various code blocks in Float and Double would be better as snippets.
>
> Joe Darcy has updated the pull request incrementally with two additional
> commits since the last revision:
>
> - Use idiom for shorter lines
> - Respond to review fee
On Mon, 27 Dec 2021 13:43:12 GMT, Markus KARG wrote:
>> Implementation of JDK-8279283
>
> Markus KARG has updated the pull request incrementally with one additional
> commit since the last revision:
>
> fixed missing BufferedInputStream
Can this PR be closed or returned to daft?
---
On Sun, 5 Jun 2022 21:19:37 GMT, Joe Darcy wrote:
> Various code blocks in Float and Double would be better as snippets.
One other thing you could do is link Pattern.matches in the snippet to the
matches method.
-
Marked as reviewed by alanb (Reviewer).
PR: https://git.openjdk.ja
On Thu, 19 May 2022 13:05:54 GMT, Alan Bateman wrote:
> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>
> This is a non-final API that provides a gentle on-ramp to structure a task as
> a family of concurrent subtasks, and to coordinate the subtas
On Fri, 3 Jun 2022 16:48:46 GMT, Naoto Sato wrote:
> The code path calls `String.getBytesNoRepl()`, but it blindly replaces
> unmappable characters with replacements if the encoder is an `ArrayEncoder`.
> Changed only to do so if `doReplace` is `true` in
> `String.encodeWithEncoder()`.
Marked
On Thu, 2 Jun 2022 18:57:36 GMT, Mandy Chung wrote:
> 8287746: ProblemList jni/nullCaller/NullCallerTest.java
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/9002
> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>
> This is a non-final API that provides a gentle on-ramp to structure a task as
> a family of concurrent subtasks, and to coordinate the subtasks as a unit.
Alan Bateman has updated the pull request with
On Sun, 29 May 2022 14:46:39 GMT, Alan Bateman wrote:
> This patch adds an alternative virtual thread implementation where each
> virtual thread is backed by an OS thread. It doesn't scale but it can be used
> by ports that don't have continuations support in the VM. Aside
On Wed, 1 Jun 2022 06:26:23 GMT, David Holmes wrote:
>> Alan Bateman has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains 11 commits:
>>
>> - Fixed another typo in comment
>> - Merge
>> -
On Wed, 1 Jun 2022 08:56:38 GMT, Raffaello Giulietti
wrote:
>> src/java.base/share/classes/jdk/internal/math/MathUtils.java line 38:
>>
>>> 36: *
>>> 37: * Giulietti, "The Schubfach way to render doubles",
>>> 38: *
>>> https://drive.google.com/file/d/1gp5xv4CAa78SVgCeWf
ption. A follow-up change would be to add "@requires
> vm.continuations" to the ~70 serviceability/jvmti/vthread that run with
> preview features enabled.
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The pull request now cont
On Tue, 31 May 2022 17:03:54 GMT, Aleksey Shipilev wrote:
> I expected this change to fix the broken ARM32 port, but it doesn't work.
There is work required to get the arm32 port working again, currently tracked
as JDK-828636 but there may be further issues beyond that.
-
PR: http
ption. A follow-up change would be to add "@requires
> vm.continuations" to the ~70 serviceability/jvmti/vthread that run with
> preview features enabled.
Alan Bateman has updated the pull request incrementally with one additional
commit since the last revision:
Allow
On Mon, 30 May 2022 13:20:17 GMT, Aleksey Shipilev wrote:
> [JDK-8287137](https://bugs.openjdk.java.net/browse/JDK-8287137) added a bunch
> of tests into problemlist. Those lists basically exclude every test that runs
> with --enable-preview.
> [JDK-8287437](https://bugs.openjdk.java.net/brows
On Sun, 29 May 2022 14:46:39 GMT, Alan Bateman wrote:
> This patch adds an alternative virtual thread implementation where each
> virtual thread is backed by an OS thread. It doesn't scale but it can be used
> by ports that don't have continuations support in the VM. Aside
This patch adds an alternative virtual thread implementation where each virtual
thread is backed by an OS thread. It doesn't scale but it can be used by ports
that don't have continuations support in the VM. Aside from scalability, the
lack of continuations support means:
1. JVM TI is not suppo
> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>
> This is a non-final API that provides a gentle on-ramp to structure a task as
> a family of concurrent subtasks, and to coordinate the subtasks as a unit.
Alan Bateman has updated the pull request with
On Mon, 30 May 2022 05:29:01 GMT, Alan Bateman wrote:
>> More practically.
>> This PR has a noticeable negative effect - it increases the size of
>> InputStream objects. Moreover, it increases the size of InputStream
>> subclasses which has own skip() implementa
On Mon, 30 May 2022 00:10:50 GMT, Tim Prinzing wrote:
>> Created a test at test/jdk/jdk/nullCaller called NullCallerTest that creates
>> a test module with some resources in it for the actual tests that occur at
>> the native level. The native part was switched to c++ instead of c to make
>> i
On Sat, 28 May 2022 03:31:30 GMT, Yasumasa Suenaga wrote:
> We found using `0` as `NULL` in java_md_common.c . `0` is not a pointer, so
> we should use `NULL` where we want to handle it.
>
> https://github.com/openjdk/jdk/pull/8646#discussion_r882294076
>
> Also I found using `0` as NUL char i
On Sun, 29 May 2022 18:15:52 GMT, Sergey Kuksenko wrote:
> 5. skip() is not implemented, when not-so-trivial implementation is possible
> (9 classes):
For the low-level streams (e.g. connected to socket) then it would be common to
see them wrapped by buffered streams. So it might not be worth
On Sat, 30 Apr 2022 08:56:23 GMT, Andrey Turbanov wrote:
> The method `java.util.zip.ZipFile.Source#get` could be improved by usage of
> `Map.putIfAbsent` instead of separate `containsKey`/`get`/`put` calls. We
> known that HashMap `java.util.zip.ZipFile.Source#files` can contain only
> non-nu
> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>
> This is a non-final API that provides a gentle on-ramp to structure a task as
> a family of concurrent subtasks, and to coordinate the subtasks as a unit.
Alan Bateman has updated the pull request i
On Tue, 24 May 2022 10:52:07 GMT, Maurizio Cimadamore
wrote:
>> Alan Bateman 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 conta
On Tue, 24 May 2022 10:48:02 GMT, Maurizio Cimadamore
wrote:
>> More generally, I see that you used `{@code ... }` in a lot of places where
>> `{@link ... }` could also be used. In some of those places (like this one)
>> where there is a clear cross-reference, I think `@link` could be
>> pref
> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>
> This is a non-final API that provides a gentle on-ramp to structure a task as
> a family of concurrent subtasks, and to coordinate the subtasks as a unit.
Alan Bateman has updated the pull request with
On Tue, 24 May 2022 04:18:44 GMT, Joe Darcy wrote:
>> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>>
>> This is a non-final API that provides a gentle on-ramp to structure a task
>> as a family of concurrent subtasks, and to coordinate the subtasks as a unit.
>
>
On Mon, 23 May 2022 21:09:24 GMT, Maurizio Cimadamore
wrote:
>> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>>
>> This is a non-final API that provides a gentle on-ramp to structure a task
>> as a family of concurrent subtasks, and to coordinate the subtasks as a
On Mon, 23 May 2022 13:11:29 GMT, Daniel Fuchs wrote:
>> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>>
>> This is a non-final API that provides a gentle on-ramp to structure a task
>> as a family of concurrent subtasks, and to coordinate the subtasks as a unit.
>
On Tue, 24 May 2022 07:01:52 GMT, Aleksey Shipilev wrote:
> Please approve, @AlanBateman, @mcimadamore and others?
Okay with me.
-
PR: https://git.openjdk.java.net/jdk/pull/8843
On Tue, 24 May 2022 09:26:44 GMT, Maurizio Cimadamore
wrote:
> This patch tweaks the foreign API to use the newly added
> `WrongThreadException` instead of `IllegalStateException` to report
> confinement errors.
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.j
On Mon, 23 May 2022 19:47:33 GMT, Lance Andersen wrote:
> Hi all,
>
> This PR addresses the performance issue that is described in JDK-8287162.
>
> With this fix, the ZipFileSystem methods: initOwner, initGroup, and
> initPermissions will not be invoked unless enablePosixFileAttributes=true.
On Mon, 23 May 2022 15:18:10 GMT, Aleksey Shipilev wrote:
>> test/jdk/ProblemList.txt line 894:
>>
>>> 892:
>>> java/util/stream/test/org/openjdk/tests/java/util/stream/CollectionAndMapModifyStreamTest.java
>>> 8286642 generic-i586
>>> 893:
>>> java/util/stream/test/org/openjdk/tests/java/ut
On Mon, 23 May 2022 12:28:30 GMT, Aleksey Shipilev wrote:
> [JDK-8284161](https://bugs.openjdk.java.net/browse/JDK-8284161) broke a lot
> of x86_32 code. The x86_32 porting is done under
> [JDK-8286642](https://bugs.openjdk.java.net/browse/JDK-8286642). Meanwhile,
> we can problemlist the fail
On Sat, 21 May 2022 14:09:59 GMT, ExE Boss wrote:
>> This is the implementation of JEP 428: Structured Concurrency (Incubator).
>>
>> This is a non-final API that provides a gentle on-ramp to structure a task
>> as a family of concurrent subtasks, and to coordinate the subtasks as a unit.
>
> s
This is the implementation of JEP 428: Structured Concurrency (Incubator).
This is a non-final API that provides a gentle on-ramp to structure a task as a
family of concurrent subtasks, and to coordinate the subtasks as a unit.
-
Commit messages:
- @ignore StructuredThreadDumpTest
On Sun, 22 May 2022 05:58:25 GMT, Christian Stein wrote:
> Commit
> https://github.com/openjdk/jdk/commit/655500a4f5e3abcff176599604deceefb6ca6640
> for issue [JDK-8286654](https://bugs.openjdk.java.net/browse/JDK-8286654)
> added an optional description accessor on the `ToolProvider` interfac
On Mon, 23 May 2022 07:29:56 GMT, Adam Sotona wrote:
> Sorry, I wrongly assumed it was `java.compiler` (the library module), but
> this is about `jdk.compiler` (the tool)... Nevertheless, the same question
> applies: Does `jdk.compiler` have any meaningful functionality even without
> `jdk.zip
On Sun, 22 May 2022 05:58:25 GMT, Christian Stein wrote:
> Commit
> https://github.com/openjdk/jdk/commit/655500a4f5e3abcff176599604deceefb6ca6640
> for issue [JDK-8286654](https://bugs.openjdk.java.net/browse/JDK-8286654)
> added an optional description accessor on the `ToolProvider` interfac
On Wed, 18 May 2022 11:10:58 GMT, Aleksey Shipilev wrote:
> It would be beneficial to bring over the Loom-specific test groups from the
> loom repo to aid development/porting work.
>
> https://github.com/openjdk/loom/blob/fibers/test/jdk/TEST.groups#L97-L108
> https://github.com/openjdk/loom/bl
On Tue, 17 May 2022 11:15:19 GMT, Alan Bateman wrote:
> This is a test fix. ThreadAPI.testGetStackTrace3 tests Thread::getStackTrace
> on a thread doing a selection operation. The test is not reliable as it
> expects to see the "select" method in the stack trace after wa
This is a test fix. ThreadAPI.testGetStackTrace3 tests Thread::getStackTrace on
a thread doing a selection operation. The test is not reliable as it expects to
see the "select" method in the stack trace after waiting 200ms. The test is
changed to poll the stack trace so that it's no longer depen
On Mon, 16 May 2022 17:29:16 GMT, Joe Darcy wrote:
>> Make the javadoc in the InputStream and OutputStream subclasses in core libs
>> DRY-er by use of inheritDoc. (Any analagous changes to AudioInputStream in
>> client libs will be done another a separate bug.) When the time comes, will
>> do
On Mon, 16 May 2022 16:56:00 GMT, Joe Darcy wrote:
> I added the `@Override` annotations to methods I reviewed and/or updated; it
> was not necessarily an exhaustive process.
Yeah, there are inconsistencies as a result and I think we should go the extra
mile and add to the methods such as rea
On Sun, 15 May 2022 18:36:07 GMT, Joe Darcy wrote:
> Make the javadoc in the InputStream and OutputStream subclasses in core libs
> DRY-er by use of inheritDoc. (Any analagous changes to AudioInputStream in
> client libs will be done another a separate bug.) When the time comes, will
> do any
On Fri, 13 May 2022 07:14:30 GMT, Jaikiran Pai wrote:
>> Can I please get a review of this change that addresses
>> https://bugs.openjdk.java.net/browse/JDK-8286559?
>>
>> The commit here removes the `synchronized` on `mark` and `reset` methods of
>> `InflaterInputStream`. The `mark` method i
On Thu, 12 May 2022 20:04:33 GMT, Joe Darcy wrote:
>> While doing a CSR review of another issue, I noticed some cases in
>> InputStream and OutputStream what would benefit from being upgraded to
>> implSpec and related javadoc tags.
>>
>> The "A subclass must provide an implementation of this
On Fri, 13 May 2022 06:47:20 GMT, Jie Fu wrote:
>> test/jdk/java/foreign/TestIntrinsics.java line 48:
>>
>>> 46: * -XX:+UseShenandoahGC
>>> 47: * TestIntrinsics
>>> 48: */
>>
>> Is this needed? The parameters looks the same as the first test description
>> so if you are testing with +Sh
On Fri, 13 May 2022 02:43:55 GMT, Jie Fu wrote:
> Hi all,
>
> Some tests fail with Shenandoah GC after JDK-8282191.
> The reason is that the assert in `ShenandoahControlThread::request_gc` misses
> the case of `GCCause::_codecache_GC_threshold`.
> It would be better to fix it.
>
> Thanks.
> Be
On Fri, 13 May 2022 04:41:03 GMT, ExE Boss wrote:
>> Roger Riggs has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Updated copyrights
>> Fixed cast style to add a space after cast, (where consistent with file
>> style)
>> Improved cod
On Wed, 11 May 2022 23:08:32 GMT, Ioi Lam wrote:
> The API `jdk.internal.perf.Perf.::attach(String user, int lvmid)` is never
> used. It should be removed, and all the handling of a specified user name
> should be removed.
Marked as reviewed by alanb (Reviewer).
src/java.base/share/classes/jd
On Wed, 11 May 2022 20:40:30 GMT, Joe Darcy wrote:
> While doing a CSR review of another issue, I noticed some cases in
> InputStream and OutputStream what would benefit from being upgraded to
> implSpec and related javadoc tags.
>
> The "A subclass must provide an implementation of this metho
On Wed, 11 May 2022 16:30:41 GMT, Roger Riggs wrote:
>> PR#8599 8244681: proposes to add compiler warnings for possible lossy
>> conversions
>> From the CSR:
>>
>> "If the type of the right-hand operand of a compound assignment is not
>> assignment compatible with the type of the variable, a c
On Wed, 11 May 2022 02:43:21 GMT, Ioi Lam wrote:
>> The `mode` parameter for ` jdk.internal.perf.Perf.attach()` has never
>> supported the value `"rw"` since the source code was imported to the openjdk
>> repo more than 15 years ago. In fact HotSpot throws
>> `IllegalArgumentException` when su
On Wed, 11 May 2022 12:47:08 GMT, Jaikiran Pai wrote:
>> src/java.base/share/native/libzip/zlib/gzwrite.c line 452:
>>
>>> 450: len = strlen(next);
>>> 451: # else
>>> 452: # ifdef __APPLE__ // ignore format-nonliteral warning on macOS
>>
>> Instead of patching 3rd party code to fix a co
On Wed, 11 May 2022 08:40:21 GMT, Yasumasa Suenaga wrote:
>> I saw some compiler warnings when I tried to build OpenJDK with GCC 12.0.1
>> on Fedora 36.
>> As you can see, the warnings spreads several areas. Let me know if I should
>> separate them by area.
>>
>> * -Wstringop-overflow
>> *
On Tue, 10 May 2022 21:30:23 GMT, Sean Mullan wrote:
> > It's probably ok, but the bug report is either incomplete or I am missing
> > something. It says "This can be improved to something like: ..." but the
> > same text as is emitted now is used. Can you fix this so I have a better
> > examp
On Tue, 10 May 2022 23:01:33 GMT, Roger Riggs wrote:
>> PR#8599 8244681: proposes to add compiler warnings for possible lossy
>> conversions
>> From the CSR:
>>
>> "If the type of the right-hand operand of a compound assignment is not
>> assignment compatible with the type of the variable, a c
On Tue, 10 May 2022 04:00:29 GMT, Ioi Lam wrote:
> The `mode` parameter for ` jdk.internal.perf.Perf.attach()` has never
> supported the value `"rw"` since the source code was imported to the openjdk
> repo more than 15 years ago. In fact HotSpot throws
> `IllegalArgumentException` when such a
On Tue, 10 May 2022 04:00:29 GMT, Ioi Lam wrote:
> The `mode` parameter for ` jdk.internal.perf.Perf.attach()` has never
> supported the value `"rw"` since the source code was imported to the openjdk
> repo more than 15 years ago. In fact HotSpot throws
> `IllegalArgumentException` when such a
On Tue, 10 May 2022 16:48:30 GMT, Christoph Langer wrote:
> I think this would be OK, but would get to get someone from our security team
> to bless it.
It's print the entry name, I don't think it is leaking the file path to the zip
file.
-
PR: https://git.openjdk.java.net/jdk/pu
On Tue, 10 May 2022 14:54:39 GMT, Aleksey Shipilev wrote:
>> There are plenty of tests failing on many architectures due to
>> `--enable-preview` VM code introduced by Loom. This improvement eliminates
>> some of the redundant `--enable-preview` clauses from the Sealed Classes
>> tests, since
On Tue, 10 May 2022 14:51:52 GMT, Aleksey Shipilev wrote:
>> There are plenty of tests failing on many architectures due to
>> `--enable-preview` VM code introduced by Loom. This improvements eliminates
>> some of the redundant `--enable-preview` clauses from the Record tests,
>> since Records
On Tue, 10 May 2022 12:03:09 GMT, Aleksey Shipilev wrote:
> There are plenty of tests failing on many architectures due to
> `--enable-preview` VM code introduced by Loom. This improvements eliminates
> some of the redundant `--enable-preview` clauses from the Record tests, since
> Records hav
On Mon, 9 May 2022 22:32:57 GMT, Christoph Langer wrote:
> @LanceAndersen @AlanBateman do you think adding the entry name in the
> exception in ZipFileSystem is ok? If so, should it maybe go into a different
> patch?
It should be okay as this is the name of an entry in the zip file. It might b
On Tue, 10 May 2022 12:07:31 GMT, Aleksey Shipilev wrote:
> There are plenty of tests failing on many architectures due to
> `--enable-preview` VM code introduced by Loom. This improvement eliminates
> some of the redundant `--enable-preview` clauses from the Sealed Classes
> tests, since Seal
On Wed, 4 May 2022 08:00:08 GMT, Matthias Baesken wrote:
>> Currently we set _WIN32_WINNT at various places in the codebase; this is
>> used to target a minimum Windows version we want to support. See also for
>> more detailled information :
>> https://docs.microsoft.com/en-us/windows/win32/win
On Mon, 9 May 2022 18:17:13 GMT, Leonid Mesnik wrote:
> 8286368: Cleanup problem lists after loom integration
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/8604
On Mon, 9 May 2022 15:26:20 GMT, Brian Burkhalter wrote:
> Add missing `@since 19` tag.
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/8598
On Sun, 8 May 2022 12:08:14 GMT, Doug Lea wrote:
>> Changes ForkJoinPool.close spec and code to trap close as a no-op if called
>> on common pool
>
> Doug Lea has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Test improvements
Marked as rev
On Sun, 8 May 2022 21:57:20 GMT, Leonid Mesnik wrote:
> …resulting in failure to produce a stack when a test times out
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/8588
On Fri, 6 May 2022 11:32:25 GMT, Claes Redestad wrote:
> A few untested and unused methods in `VerifyType` which can be removed.
> (Possibly used by native JSR 292 implementations in JDK 7).
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/8570
On Sun, 8 May 2022 02:22:08 GMT, Phil Race wrote:
> I did wonder why it has security-libs as the sub-category and if the intent
> was not what we see here.
I suspect the JBS issue was initially created to to look at the usages of
getProperty in the security code but it has been extended. The i
On Fri, 8 Apr 2022 13:43:39 GMT, Alan Bateman wrote:
> This is the implementation of JEP 425: Virtual Threads (Preview).
>
> We will refresh this PR periodically to pick up changes and fixes from the
> loom repo.
>
> Most of the new mechanisms in the HotSpot VM are disab
nd Extent Locals. This is to
> make life a bit easier and avoid having to separate VM changes and juggle
> branches at this time.
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The pull request now contains 23 commits:
- Refresh 4e99b5185eef939
On Thu, 5 May 2022 16:49:07 GMT, Mark Powers wrote:
> JDK-6725221 Standardize obtaining boolean properties with defaults
src/java.base/share/classes/java/lang/reflect/AccessibleObject.java line 777:
> 775: if (!printStackPropertiesSet && VM.initLevel() >= 1) {
> 776: printSt
On Fri, 6 May 2022 06:48:46 GMT, Alan Bateman wrote:
>> This is the implementation of JEP 425: Virtual Threads (Preview).
>>
>> We will refresh this PR periodically to pick up changes and fixes from the
>> loom repo.
>>
>> Most of the new mechanisms in th
nd Extent Locals. This is to
> make life a bit easier and avoid having to separate VM changes and juggle
> branches at this time.
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The pull request now contains 21 commits:
- Refresh 6ace49bf42e5504
On Thu, 5 May 2022 17:43:58 GMT, Aleksey Shipilev wrote:
> I am sorry to be a buzzkill here, but this integration would break lots of
> platforms even when Loom functionality is not enabled/used. For example,
> running `java -version` on RISC-V runs into many issues:
> `TemplateInterpreterGene
On Wed, 4 May 2022 12:04:47 GMT, Matthias Baesken wrote:
> A couple a tests like java/lang/ProcessBuilder/Basic.java#id0.Basic_id0 and
> jdk/jshell/ExternalEditorTest.java.ExternalEditorTest try to start small
> shell scripts without #! at the first line of the script. This fails with
> error=
On Tue, 19 Apr 2022 08:40:51 GMT, Raffaello Giulietti
wrote:
> Please review these small changes to address intermittent failures, as of
> JDK-8274517.
>
> - Usage of jdk.test.lib.RandomFactory for reproducible random generation.
> - Slightly less restrictive assertion about badParallelStreamE
1 - 100 of 1316 matches
Mail list logo