On Sun, 6 Dec 2020 02:57:03 GMT, Martin Buchholz wrote:
> 8234131: Miscellaneous changes imported from jsr166 CVS 2020-12
It would be nice to get it tested with GH Actions. "Checks" tabs should have
those testing results automatically on push, but it is empty. Probably because
your fork is not
On Sun, 6 Dec 2020 02:56:34 GMT, Martin Buchholz wrote:
> 8246585: ForkJoin updates
It would be nice to get it tested with GH Actions. "Checks" tabs should have
those testing results automatically on push, but it is empty. Probably because
your fork is not configured for it. Please go to
http
On Mon, 7 Dec 2020 17:38:40 GMT, Maurizio Cimadamore
wrote:
> This simple patch updates the jaavdoc in the module-info file of
> jdk.incubator.foreign to reflect the fact that support of foreign function
> calls has been added.
Looks fine.
-
Marked as reviewed by shade (Reviewer
On Tue, 8 Dec 2020 02:40:43 GMT, Mandy Chung wrote:
>> I have reviewed all lines in the patch file with or near instances of
>> `jdk.compiler`
>
> Hi Magnus,
>
> I see the motivation of moving these build files for better identification of
> ownership. Placing them under
> `src/$MODULE/{shar
On Tue, 8 Dec 2020 08:27:16 GMT, Magnus Ihse Bursie wrote:
>> Hi Magnus,
>>
>> I see the motivation of moving these build files for better identification
>> of ownership. Placing them under
>> `src/$MODULE/{share,$OS}/data` is one option. Given that skara will
>> automatically determine app
On Tue, 8 Dec 2020 06:11:27 GMT, Martin Buchholz wrote:
>> 8234131: Miscellaneous changes imported from jsr166 CVS 2020-12
>
> Martin Buchholz has refreshed the contents of this pull request, and previous
> commits have been removed. The incremental views will show differences
> compared to the
See here:
https://github.com/mcimadamore/jdk/runs/1460615378
$ CONF=linux-x86-server-fastdebug make images run-test
TEST=java/foreign/TestSegments.java
STDERR:
Invalid maximum heap size: -Xmx4G
The specified size exceeds the maximum representable size.
Error: Could not create the Java Virtual M
Hi all,
java/foreign/TestSegments.java fails on x86_32 due to '-Xmx4G'.
The reason is that -Xmx4G is invalid maximum heap size for 32-bit platforms.
The current implimentation only supports maximum 3800M on 32-bit systems [1].
The fix just change '-Xmx4G' to '-Xmx3500M'.
Testing:
- java/foreig
On Tue, 8 Dec 2020 08:59:25 GMT, Jie Fu wrote:
> Hi all,
>
> java/foreign/TestSegments.java fails on x86_32 due to '-Xmx4G'.
> The reason is that -Xmx4G is invalid maximum heap size for 32-bit platforms.
> The current implimentation only supports maximum 3800M on 32-bit systems [1].
>
> The fix
On Tue, 8 Dec 2020 09:07:27 GMT, Aleksey Shipilev wrote:
> `-Xm3800M` could still fail on many 32-bit systems, for example where native
> libraries are located in the middle of address space. I think it is safer to
> `@require` it. In fact, I did it an hour ago here #1688 :)
The test just limi
On Mon, 7 Dec 2020 23:47:40 GMT, Mandy Chung wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a sealed
On Tue, 8 Dec 2020 08:54:41 GMT, Aleksey Shipilev wrote:
> See here:
> https://github.com/mcimadamore/jdk/runs/1460615378
>
> $ CONF=linux-x86-server-fastdebug make images run-test
> TEST=java/foreign/TestSegments.java
>
> STDERR:
> Invalid maximum heap size: -Xmx4G
> The specified size excee
On Mon, 7 Dec 2020 16:35:52 GMT, Athijegannathan Sundararajan
wrote:
> Safe URI encode logic adopted from UnixUriUtils.
This pull request has now been integrated.
Changeset: d2b66196
Author:Athijegannathan Sundararajan
URL: https://git.openjdk.java.net/jdk/commit/d2b66196
Stats:
Please review this change that eliminates the use of Reference.isEnqueued by
tests. There were three tests using it:
vmTestbase/gc/gctests/ReferencesGC/ReferencesGC.java
vmTestbase/gc/gctests/WeakReferenceGC/WeakReferenceGC.java
jdk/java/lang/ref/ReferenceEnqueue.java
In each of them, some combi
On Tue, 8 Dec 2020 09:22:05 GMT, Jie Fu wrote:
> > `-Xm3800M` could still fail on many 32-bit systems, for example where
> > native libraries are located in the middle of address space. I think it is
> > safer to `@require` it. In fact, I did it an hour ago here #1688 :)
>
> The test just limi
On Tue, 8 Dec 2020 06:30:28 GMT, Martin Buchholz wrote:
>> 8254350: CompletableFuture.get may swallow InterruptedException
>
> Martin Buchholz has refreshed the contents of this pull request, and previous
> commits have been removed. The incremental views will show differences
> compared to the
On Tue, 8 Dec 2020 10:19:22 GMT, Daniel Fuchs wrote:
>> Martin Buchholz has refreshed the contents of this pull request, and
>> previous commits have been removed. The incremental views will show
>> differences compared to the previous content of the PR.
>
> test/jdk/java/util/concurrent/Comple
On Mon, 7 Dec 2020 18:22:33 GMT, Daniel Fuchs wrote:
>> This change applies a stricter semantic distinction of 'type' versus 'class
>> and interface'. This is based on the JLS changes described in the
>> "Consistent Class and Interface Terminology" document:
>> https://download.java.net/java/e
On Tue, 8 Dec 2020 10:17:48 GMT, Aleksey Shipilev wrote:
> > > `-Xm3800M` could still fail on many 32-bit systems, for example where
> > > native libraries are located in the middle of address space. I think it
> > > is safer to `@require` it. In fact, I did it an hour ago here #1688 :)
> >
>
On Tue, 8 Dec 2020 08:54:41 GMT, Aleksey Shipilev wrote:
> See here:
> https://github.com/mcimadamore/jdk/runs/1460615378
>
> $ CONF=linux-x86-server-fastdebug make images run-test
> TEST=java/foreign/TestSegments.java
>
> STDERR:
> Invalid maximum heap size: -Xmx4G
> The specified size excee
On Tue, 8 Dec 2020 08:59:25 GMT, Jie Fu wrote:
> Hi all,
>
> java/foreign/TestSegments.java fails on x86_32 due to '-Xmx4G'.
> The reason is that -Xmx4G is invalid maximum heap size for 32-bit platforms.
> The current implimentation only supports maximum 3800M on 32-bit systems [1].
>
> The fix
This change have been already reviewed by Mandy, Alan and David.
Now, the PR approval is needed.
The push was postponed because the CSR was not approved at that time (it is
now):
https://bugs.openjdk.java.net/browse/JDK-8248189
Investigation of existing popular java agents was requested by Joe.
On Tue, 8 Dec 2020 08:53:38 GMT, Alan Bateman wrote:
>> Martin Buchholz has refreshed the contents of this pull request, and
>> previous commits have been removed. The incremental views will show
>> differences compared to the previous content of the PR.
>
> src/java.base/share/classes/java/uti
On Tue, 8 Dec 2020 10:35:20 GMT, Alan Bateman wrote:
>> test/jdk/java/util/concurrent/CompletableFuture/LostInterrupt.java line 49:
>>
>>> 47:
>>> 48: public static void main(String[] args) throws Exception {
>>> 49: ThreadLocalRandom rnd = ThreadLocalRandom.current();
>>
>> Hi Mar
On 12/8/20 3:56 AM, Alan Bateman wrote:
1558: break;
1559: }
1560: */
Is this meant to be commented out?
Yes, but It should be marked as a possible improvement, not yet
committed. While this (prestarting) would improve performance in some
scenarios
Dear all,
The class java.util.Optional could be considered a collection with a limit
on how many values it can hold, minimum 0, maximum 1.
Likewise, any class implementing the regular java.util.Collection has a
minimum, 0 and a maximum > 1.
While Optional does not implement Collection, it could b
On Mon, 7 Dec 2020 17:38:40 GMT, Maurizio Cimadamore
wrote:
> This simple patch updates the jaavdoc in the module-info file of
> jdk.incubator.foreign to reflect the fact that support of foreign function
> calls has been added.
This pull request has now been integrated.
Changeset: a7080247
A
On Tue, 8 Dec 2020 08:32:28 GMT, Magnus Ihse Bursie wrote:
>> @mlchung If you don't have any strong preference, I implore you to accept
>> this change. I **vastly** prefer to move the data out of `make`!
>>
>> This is not just about Skara tooling. When working with make files, autoconf
>> and
- Mail original -
> De: "Dave Franken"
> À: "core-libs-dev"
> Envoyé: Mardi 8 Décembre 2020 13:17:25
> Objet: Has it been considered to add inverse methods to collections which are
> in Optional?
> Dear all,
Hi Dave,
>
> The class java.util.Optional could be considered a collection w
On Tue, 8 Dec 2020 10:41:06 GMT, Julia Boes wrote:
>> src/java.base/share/classes/java/lang/Enum.java line 62:
>>
>>> 60: * java.util.EnumMap map} implementations are available.
>>> 61: *
>>> 62: * @param The enum class subclass
>>
>> I wonder about this one, given that `` is a type. It sou
On 2020-12-08 00:30, Magnus Ihse Bursie wrote:
On Tue, 8 Dec 2020 02:40:43 GMT, Mandy Chung wrote:
I have reviewed all lines in the patch file with or near instances of
`jdk.compiler`
Hi Magnus,
I see the motivation of moving these build files for better identification of
ownership. Pl
On Tue, 8 Dec 2020 14:07:08 GMT, Harold Seigel wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a seal
> Please review this fix for JDK-8256867. This change no longer throws a
> ClassFormatError exception when loading a class whose PermittedSubclasses
> attribute is empty (contains no classes). Instead, the class is treated as a
> sealed class which cannot be extended nor implemented. This new
On Tue, 8 Dec 2020 00:09:34 GMT, Mandy Chung wrote:
>> src/hotspot/share/prims/jvm.cpp line 2130:
>>
>>> 2128: JvmtiVMObjectAllocEventCollector oam;
>>> 2129: Array* subclasses = ik->permitted_subclasses();
>>> 2130: int length = subclasses == NULL ? 0 : subclasses->length();
>>
>>
On Tue, 8 Dec 2020 09:30:59 GMT, Chris Hegarty wrote:
>> src/java.base/share/classes/java/lang/Class.java line 4396:
>>
>>> 4394: * is unspecified. If this {@code Class} object represents a
>>> primitive type,
>>> 4395: * {@code void}, an array type, or a class or interface that is
>
On Tue, 8 Dec 2020 14:10:26 GMT, Harold Seigel wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a seal
On Tue, 8 Dec 2020 14:37:52 GMT, Chris Hegarty wrote:
>> Harold Seigel has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8256867: Classes with empty PermittedSubclasses attribute cannot be
>> extended
>
> src/java.base/share/classes/java/
> Please review this fix for JDK-8256867. This change no longer throws a
> ClassFormatError exception when loading a class whose PermittedSubclasses
> attribute is empty (contains no classes). Instead, the class is treated as a
> sealed class which cannot be extended nor implemented. This new
On Tue, 8 Dec 2020 03:08:59 GMT, Alexander Matveev wrote:
>
>
> Looks like test failed due to:
> [Fatal Error] b.wxl:3:1: XML document structures must start and end within
> the same entity.
> So, I am not sure how increased repeat will help. Do we know why it failed to
> parse XML?
If you s
On Tue, 8 Dec 2020 14:57:26 GMT, Harold Seigel wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a seal
> increase retries and timeout increasing in runMsiexecWithRetries
Andy Herrick has updated the pull request incrementally with 18 additional
commits since the last revision:
- 8256149: Weird AST structure for incomplete member select
Reviewed-by: vromero
- 8257194: Add 'foreign linker
On Tue, 8 Dec 2020 00:48:55 GMT, Stuart Marks wrote:
>> The algorithm can be well defined for minGrowth and prefGrowth == 0 without
>> extra checks or exceptions with a careful look at the inequality checks.
>> For example, as currently coded, if both are zero, it returns
>> SOFT_MAX_ARRAY_LENG
On Tue, 8 Dec 2020 06:14:35 GMT, Stuart Marks wrote:
>> This rewrites the doc of ArraysSupport.newLength, adds detail to the
>> exception message, and adds a test. In addition to some renaming and a bit
>> of refactoring of the actual code, I also made two changes of substance to
>> the code:
On Tue, 8 Dec 2020 12:15:38 GMT, Alan Bateman wrote:
>> Also, to clarify, for me there is a fundamental difference between
>> `src/$MODULE` and `make/modules/$MODULE`. The former is the home of files
>> that are part of the module, owned by the content team, and the `$MODULE`
>> part is essent
On Tue, 8 Dec 2020 12:15:38 GMT, Alan Bateman wrote:
>> Also, to clarify, for me there is a fundamental difference between
>> `src/$MODULE` and `make/modules/$MODULE`. The former is the home of files
>> that are part of the module, owned by the content team, and the `$MODULE`
>> part is essent
> This change applies a stricter semantic distinction of 'type' versus 'class
> and interface'. This is based on the JLS changes described in the "Consistent
> Class and Interface Terminology" document:
> https://download.java.net/java/early_access/jdk16/docs/specs/class-terminology-jls.html.
>
On Tue, 8 Dec 2020 15:25:47 GMT, Weijun Wang wrote:
>> The mapping and nr tables, and the *-X.java.template files in
>> make/data/charsetmapping are used to generate source code for the java.base
>> and jdk.charsets modules. The stdcs-$OS files configure the package and
>> module that each cha
On Thu, 3 Dec 2020 23:40:40 GMT, Joe Wang wrote:
>> Marius Volkhart has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - fixup! Fix: javax.xml.stream.XMLEventReader produces incorrect
>> START_DOCUMENT event
>> - fixup! Fix: javax.xml.str
On Tue, 8 Dec 2020 14:56:20 GMT, Andy Herrick wrote:
>> Looks like test failed due to:
>> [Fatal Error] b.wxl:3:1: XML document structures must start and end within
>> the same entity.
>> So, I am not sure how increased repeat will help. Do we know why it failed
>> to parse XML?
>
>>
>>
>> Lo
> The default implementation of javax.xml.stream.XMLEventReader produced a
> StartDocument event that always indicated that the "standalone" attribute was
> set.
>
> The root cause of this was that the
> com.sun.xml.internal.stream.events.XMLEventAllocatorImpl always set the
> "standalone" att
On Tue, 8 Dec 2020 16:21:09 GMT, Julia Boes wrote:
>> This change applies a stricter semantic distinction of 'type' versus 'class
>> and interface'. This is based on the JLS changes described in the
>> "Consistent Class and Interface Terminology" document:
>> https://download.java.net/java/ear
On Tue, 8 Dec 2020 16:21:09 GMT, Julia Boes wrote:
>> This change applies a stricter semantic distinction of 'type' versus 'class
>> and interface'. This is based on the JLS changes described in the
>> "Consistent Class and Interface Terminology" document:
>> https://download.java.net/java/ear
Updates to the specifications of Double.{equals, compareTo} to explain more
explicitly why the obvious wrappers around "==" and "<"/"==" are not sufficient
for floating-point values.
Once the wording is worked out, I'll replicate it for the analogous methods on
Float.
-
Commit mes
On Fri, 4 Dec 2020 20:48:01 GMT, Daniel Fuchs wrote:
>> test/jdk/sun/net/www/protocol/jar/MultiReleaseJarURLConnection.java line 80:
>>
>>> 78: creator.buildSignedMultiReleaseJar();
>>> 79:
>>> 80: server = new
>>> SimpleHttpServer(TESTCONTEXT,System.getProperty("user.dir", "."
> jaxp.library.SimpleHttpServer
> https://bugs.openjdk.java.net/browse/JDK-8212035
Mahendra Chhipa has updated the pull request incrementally with one additional
commit since the last revision:
Implemented the review comments.
-
Changes:
- all: https://git.openjdk.java.net/jdk/
On Fri, 4 Dec 2020 20:45:24 GMT, Daniel Fuchs wrote:
>> Mahendra Chhipa has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Implemented the review comments.
>
> test/lib/jdk/test/lib/net/SimpleHttpServer.java line 95:
>
>> 93: retur
On 08-Dec-20 5:40, Mandy Chung wrote:
Thanks David. I was about to create one.
This is indeed a gap in injecting this trampoline class for supporting
@CallerSensitive methods. The InjectedInvoker ensures that the
InjectedInvoker class has the same class loader as the lookup class.
W.r.t. y
> Start of JDK 17 updates.
Joe Darcy 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 12 additional commits since the
last revision:
- Merge branch 'm
On Tue, 8 Dec 2020 16:29:49 GMT, Joe Darcy wrote:
> Updates to the specifications of Double.{equals, compareTo} to explain more
> explicitly why the obvious wrappers around "==" and "<"/"==" are not
> sufficient for floating-point values.
>
> Once the wording is worked out, I'll replicate it f
On Tue, 8 Dec 2020 08:54:41 GMT, Aleksey Shipilev wrote:
> See here:
> https://github.com/mcimadamore/jdk/runs/1460615378
>
> $ CONF=linux-x86-server-fastdebug make images run-test
> TEST=java/foreign/TestSegments.java
>
> STDERR:
> Invalid maximum heap size: -Xmx4G
> The specified size excee
On Mon, 7 Dec 2020 20:30:07 GMT, Andrey Turbanov
wrote:
> 6882207: Convert javap to use diamond operator internally
Nice; thanks
-
Marked as reviewed by jjg (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1677
On Tue, 8 Dec 2020 14:57:26 GMT, Harold Seigel wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a seal
On Tue, 8 Dec 2020 09:52:51 GMT, Kim Barrett wrote:
> Please review this change that eliminates the use of Reference.isEnqueued by
> tests. There were three tests using it:
>
> vmTestbase/gc/gctests/ReferencesGC/ReferencesGC.java
> vmTestbase/gc/gctests/WeakReferenceGC/WeakReferenceGC.java
> jd
On Tue, 8 Dec 2020 12:15:38 GMT, Alan Bateman wrote:
>> Also, to clarify, for me there is a fundamental difference between
>> `src/$MODULE` and `make/modules/$MODULE`. The former is the home of files
>> that are part of the module, owned by the content team, and the `$MODULE`
>> part is essent
On Tue, 8 Dec 2020 17:18:20 GMT, Mandy Chung wrote:
>> Harold Seigel has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8256867: Classes with empty PermittedSubclasses attribute cannot be
>> extended
>
> src/java.base/share/classes/java/la
On Tue, 8 Dec 2020 16:29:49 GMT, Joe Darcy wrote:
> Updates to the specifications of Double.{equals, compareTo} to explain more
> explicitly why the obvious wrappers around "==" and "<"/"==" are not
> sufficient for floating-point values.
>
> Once the wording is worked out, I'll replicate it f
On Tue, 8 Dec 2020 17:38:12 GMT, Chris Hegarty wrote:
>> src/java.base/share/classes/java/lang/Class.java line 4399:
>>
>>> 4397: * that is {@link #isSealed()} returns {@code false}, then this
>>> method returns {@code null}.
>>> 4398: * Conversely, if {@link #isSealed()} returns {@co
On Fri, 4 Dec 2020 18:23:55 GMT, Brent Christian wrote:
> Please review this fix for the intermittently-failing
> java/lang/ClassLoader/nativeLibrary/NativeLibraryTest.java.
>
> The change replaces System.gc()+sleep() with the more robust gcAwait()
> mechanic used elsewhere in the test base, [
On Tue, 8 Dec 2020 16:19:05 GMT, Magnus Ihse Bursie wrote:
>> Is there a plan to move make/jdk/src/classes/build/tools/intpoly into
>> java.base as well?
>>
>> Update: I see all subdirs in tools are still there.
>
> @wangweij Moving build tools is a related, but separate, question, which is
>
On Tue, 8 Dec 2020 18:16:15 GMT, Mandy Chung wrote:
>> @wangweij Moving build tools is a related, but separate, question, which is
>> addressed by https://bugs.openjdk.java.net/browse/JDK-8241463.
>>
>> I send out a review earlier this year (see
>> https://mail.openjdk.java.net/pipermail/build
On Tue, 8 Dec 2020 14:57:26 GMT, Harold Seigel wrote:
>> Please review this fix for JDK-8256867. This change no longer throws a
>> ClassFormatError exception when loading a class whose PermittedSubclasses
>> attribute is empty (contains no classes). Instead, the class is treated as
>> a seal
On Tue, 8 Dec 2020 17:10:30 GMT, Joe Darcy wrote:
>> Start of JDK 17 updates.
>
> Joe Darcy 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 12 addi
On Tue, 8 Dec 2020 18:24:21 GMT, Mandy Chung wrote:
>> Harold Seigel has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8256867: Classes with empty PermittedSubclasses attribute cannot be
>> extended
>
> Marked as reviewed by mchung (Revie
On Thu, 3 Dec 2020 06:50:11 GMT, Anton Kozlov wrote:
>> Filed https://bugs.openjdk.java.net/browse/JDK-8257633
>
> Thanks for taking care of those issues. To be clear, there is no real need to
> bump the version for this PR, 10.9 is fine. This PR just proposes another way
> to implement the wor
On Tue, 8 Dec 2020 17:33:16 GMT, Alan Bateman wrote:
>> The mapping and nr tables, and the *-X.java.template files in
>> make/data/charsetmapping are used to generate source code for the java.base
>> and jdk.charsets modules. The stdcs-$OS files configure the package and
>> module that each ch
On Tue, 8 Dec 2020 17:17:54 GMT, Aditya Mandaleeka wrote:
>> See here:
>> https://github.com/mcimadamore/jdk/runs/1460615378
>>
>> $ CONF=linux-x86-server-fastdebug make images run-test
>> TEST=java/foreign/TestSegments.java
>>
>> STDERR:
>> Invalid maximum heap size: -Xmx4G
>> The specified
> `Reference::isEnqueued` method was never implemented as it was initially
> specified since 1.2. The specification says that it tests if this reference
> object has been enqueued, but the long-standing behavior is to test if this
> reference object is in its associated queue, only reflecting th
On Tue, 8 Dec 2020 16:25:25 GMT, Marius Volkhart
wrote:
>> The default implementation of javax.xml.stream.XMLEventReader produced a
>> StartDocument event that always indicated that the "standalone" attribute
>> was set.
>>
>> The root cause of this was that the
>> com.sun.xml.internal.strea
On Tue, 8 Dec 2020 17:33:16 GMT, Alan Bateman wrote:
>> The mapping and nr tables, and the *-X.java.template files in
>> make/data/charsetmapping are used to generate source code for the java.base
>> and jdk.charsets modules. The stdcs-$OS files configure the package and
>> module that each ch
> This is an update to javac and javadoc, to introduce support for Preview
> APIs, and generally improve javac and javadoc behavior to more closely adhere
> to JEP 12.
>
> The notable changes are:
>
> * adding support for Preview APIs (javac until now supported primarily only
> preview langua
On Tue, 8 Dec 2020 18:53:30 GMT, Mandy Chung wrote:
>> `Reference::isEnqueued` method was never implemented as it was initially
>> specified since 1.2. The specification says that it tests if this reference
>> object has been enqueued, but the long-standing behavior is to test if this
>> refer
On Tue, Dec 8, 2020 at 12:05 AM Aleksey Shipilev
wrote:
> On Sun, 6 Dec 2020 02:56:34 GMT, Martin Buchholz
> wrote:
>
> > 8246585: ForkJoin updates
>
> It would be nice to get it tested with GH Actions. "Checks" tabs should
> have those testing results automatically on push, but it is empty. Pro
On Tue, 8 Dec 2020 19:35:39 GMT, Alan Bateman wrote:
>> Mandy Chung 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 four additional
>> commits s
On Tue, 8 Dec 2020 19:10:37 GMT, Joe Wang wrote:
>> Marius Volkhart has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - fixup! Fix: javax.xml.stream.XMLEventReader produces incorrect
>> START_DOCUMENT event
>> - fixup! Fix: javax.xml.str
OK, rollback committed to CVS:
--- src/main/java/util/concurrent/ThreadPoolExecutor.java 27 Nov 2020
17:42:00 - 1.194
+++ src/main/java/util/concurrent/ThreadPoolExecutor.java 8 Dec 2020
20:31:54 -
@@ -1522,13 +1522,11 @@
// As a heuristic, prestart enough new workers (up to n
> `Reference::isEnqueued` method was never implemented as it was initially
> specified since 1.2. The specification says that it tests if this reference
> object has been enqueued, but the long-standing behavior is to test if this
> reference object is in its associated queue, only reflecting th
On Wed, 4 Nov 2020 14:01:53 GMT, Marius Volkhart
wrote:
> The default implementation of javax.xml.stream.XMLEventReader produced a
> StartDocument event that always indicated that the "standalone" attribute was
> set.
>
> The root cause of this was that the
> com.sun.xml.internal.stream.even
On Mon, Dec 7, 2020 at 11:56 PM Alan Bateman wrote:
>
> > 37: // TODO: Rewrite as a CompletableFuture tck test ?
>
> Do you want the "TODO" in the commit?
>
> Yes!
On Tue, Dec 8, 2020 at 3:54 AM Daniel Fuchs wrote:
>
> > RandomFactory is probably overkill here as the test just needs to
> exercise untimed and timed get. So just a random boolean rather than a wide
> range of random values.
>
> OK.
>
There's a conflict between the desires to do more thorough
On Thu, 3 Dec 2020 09:22:18 GMT, Jan Lahoda wrote:
>> Adding support for record classes in the historical data for ct.sym. This
>> includes a few changes not strictly needed for the change:
>> -updating and moving tests into test/langtools, so that it is easier to run
>> them.
>> -fixing Record
> 8234131: Miscellaneous changes imported from jsr166 CVS 2020-12
Martin Buchholz has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains one commit:
JDK-8234131
-
Changes: https://git.openjdk.java.net/jdk/pull/1647/files
Web
On Sat, 5 Dec 2020 01:46:31 GMT, Dan Smith wrote:
> Integration of [JEP 390](https://bugs.openjdk.java.net/browse/JDK-8249100).
>
> Development has been broken into 5 tasks, each with its own JBS issue:
> - [JDK-8252180](https://bugs.openjdk.java.net/browse/JDK-8252180): Deprecate
> wrapper cla
On Thu, 19 Nov 2020 16:49:30 GMT, Mark Reinhold wrote:
> Please review this implementation of JEP 396
> (https://openjdk.java.net/jeps/396).
> Alan Bateman is the original author; I’ve credited him in the commit metadata.
> Passes tiers 1-3 on {linux,macos,windows}-x64 and linux-aarch64.
This p
On Tue, 8 Dec 2020 17:59:41 GMT, Stuart Marks wrote:
>> Updates to the specifications of Double.{equals, compareTo} to explain more
>> explicitly why the obvious wrappers around "==" and "<"/"==" are not
>> sufficient for floating-point values.
>>
>> Once the wording is worked out, I'll replic
On Tue, 8 Dec 2020 17:10:30 GMT, Joe Darcy wrote:
>> Start of JDK 17 updates.
>
> Joe Darcy 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 12 addi
This is a follow-up on JDK-8255342 that removes non-specified JVM checks on
classes with Record attributes. That introduces a regression in
`InstanceKlass::is_record` that returns true on a non-record class which has
`RecordComponents` attribute present. This causes unexpected semantics in
`
On Sat, 5 Dec 2020 01:46:31 GMT, Dan Smith wrote:
> Integration of [JEP 390](https://bugs.openjdk.java.net/browse/JDK-8249100).
>
> Development has been broken into 5 tasks, each with its own JBS issue:
> - [JDK-8252180](https://bugs.openjdk.java.net/browse/JDK-8252180): Deprecate
> wrapper cla
This is similar issue we used to have for hdiutil create/detach, but for
"convert". Added same workaround to repeat command. I also added repeat for
"udifrez" command just in case.
-
Commit messages:
- 8238781: [macos] jpackage tests failed due to "hdiutil: convert failed" in
vari
> Start of JDK 17 updates.
Joe Darcy 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 additional commits since the
last revision:
- Get in JEP 390
- Mail original -
> De: "Mandy Chung"
> À: "core-libs-dev" , "hotspot-dev"
>
> Envoyé: Mardi 8 Décembre 2020 23:57:39
> Objet: RFR: 8257596: Clarify trusted final fields for record classes
Hi Mandy,
> This is a follow-up on JDK-8255342 that removes non-specified JVM checks on
> classes
1 - 100 of 108 matches
Mail list logo
