Allow building OpenJDK with UBSan. Currently the build fails when optimizing
the image due to lots of undefined behavior (it invokes the built JVM). Follow
up PRs will either replace the undefined behavior with well defined behavior or
suppress errors which are intentional. The goal is to make O
On Thu, 8 Dec 2022 12:37:17 GMT, Sergey Tsypanov wrote:
> Newer version of IntelliJ IDEA introduces new
> [inspection](https://youtrack.jetbrains.com/issue/IDEA-301797/IDEA-should-report-redundant-array-length-check-in-certain-cases)
> detecting redundant array length check in snippets like
>
On Thu, 8 Dec 2022 12:37:17 GMT, Sergey Tsypanov wrote:
> Newer version of IntelliJ IDEA introduces new
> [inspection](https://youtrack.jetbrains.com/issue/IDEA-301797/IDEA-should-report-redundant-array-length-check-in-certain-cases)
> detecting redundant array length check in snippets like
>
On Thu, 8 Dec 2022 18:10:12 GMT, Joe Darcy wrote:
> Should be an innocuous change of a private field to final; I'll run an
> internal round of sanity tests before any push to make sure there isn't any
> unexpected interaction.
This pull request has now been integrated.
Changeset: 7f9c6ce3
Aut
On Thu, 8 Dec 2022 12:37:17 GMT, Sergey Tsypanov wrote:
> Newer version of IntelliJ IDEA introduces new
> [inspection](https://youtrack.jetbrains.com/issue/IDEA-301797/IDEA-should-report-redundant-array-length-check-in-certain-cases)
> detecting redundant array length check in snippets like
>
On Thu, 8 Dec 2022 12:37:17 GMT, Sergey Tsypanov wrote:
> Newer version of IntelliJ IDEA introduces new
> [inspection](https://youtrack.jetbrains.com/issue/IDEA-301797/IDEA-should-report-redundant-array-length-check-in-certain-cases)
> detecting redundant array length check in snippets like
>
On Wed, 7 Dec 2022 17:38:31 GMT, Brent Christian wrote:
>> [JDK-8290036](https://bugs.openjdk.org/browse/JDK-8290036) documented the
>> shutdown sequence, noting that calling Runtime.halt() skips the shutdown
>> sequence and immediately terminates the VM. Thus, "threads' current methods
>> do
On Tue, 6 Dec 2022 18:10:38 GMT, Naoto Sato wrote:
>> src/java.base/share/classes/java/io/Console.java line 99:
>>
>>> 97: */
>>> 98:
>>> 99: public class Console implements Flushable
>>
>> Should we perhaps `seal` this class and only `permit` `ProxyingConsole` to
>> `extend` it?
>
> Right.
> `java.io.InputStream::transferTo` could conceivably return a negative value
> if the count of bytes transferred overflows a `long`. Modify the method to
> limit the number of bytes transferred to `Long.MAX_VALUE` per invocation.
Brian Burkhalter has updated the pull request with a new target b
On Thu, 8 Dec 2022 18:10:12 GMT, Joe Darcy wrote:
> Should be an innocuous change of a private field to final; I'll run an
> internal round of sanity tests before any push to make sure there isn't any
> unexpected interaction.
Marked as reviewed by alanb (Reviewer).
-
PR: https:/
Should be an innocuous change of a private field to final; I'll run an internal
round of sanity tests before any push to make sure there isn't any unexpected
interaction.
-
Commit messages:
- JDK-8297679: InvocationTargetException field named target is not declared
fina
Changes:
On Fri, 25 Nov 2022 18:54:28 GMT, Alan Bateman wrote:
> Another small step in the multi-release/multi-year effort to remove cruft
> from Thread/ThreadGroup.
>
> java.lang.ThreadGroup.allowThreadSuspension(boolean) dates from JDK 1.1 and
> the Classic VM. The method controlled whether threads w
On Thu, 8 Dec 2022 17:15:46 GMT, Alan Bateman wrote:
>> Another small step in the multi-release/multi-year effort to remove cruft
>> from Thread/ThreadGroup.
>>
>> java.lang.ThreadGroup.allowThreadSuspension(boolean) dates from JDK 1.1 and
>> the Classic VM. The method controlled whether threa
Hello,
Okay, so it looks like the test is expected the same bit pattern to be
used for a NaN output if a NaN was used as an input.
That isn't necessarily unreasonable, but it is *not* required by the
specifications for the Math or StrictMath method, spec for Math.acos:
Returns the arc cosin
> Another small step in the multi-release/multi-year effort to remove cruft
> from Thread/ThreadGroup.
>
> java.lang.ThreadGroup.allowThreadSuspension(boolean) dates from JDK 1.1 and
> the Classic VM. The method controlled whether threads were suspended when the
> GC failed. It appears to have
On Tue, 1 Nov 2022 05:49:25 GMT, Joe Darcy wrote:
> Usual start-of-release updates. Symbol updates in initial version reflect JDK
> 20 build 21.
This pull request has now been integrated.
Changeset: 175e3d3f
Author:Joe Darcy
Committer: Jesper Wilhelmsson
URL:
https://git.openjdk.o
On Thu, 8 Dec 2022 15:57:23 GMT, Per Minborg wrote:
>> This PR proposes changing variable names and text from "session" to "scope".
>> The proposed changes are only for the FFM API and not other parts like the
>> Vector API and test classes.
>
> Per Minborg has updated the pull request incremen
> This PR proposes changing variable names and text from "session" to "scope".
> The proposed changes are only for the FFM API and not other parts like the
> Vector API and test classes.
Per Minborg has updated the pull request incrementally with one additional
commit since the last revision:
On Thu, 8 Dec 2022 14:36:52 GMT, Maurizio Cimadamore
wrote:
>> src/java.base/share/classes/jdk/internal/foreign/abi/Binding.java line 255:
>>
>>> 253: * the context's allocator is accessed.
>>> 254: */
>>> 255: public static Context ofScope() {
>>
>> I think `ofArena`
On Tue, 6 Dec 2022 14:04:44 GMT, Dmitry Samersoff
wrote:
> If the user has set LD_LIBRARY_PATH to use a particular libjvm.so, options
> from the JDK_JAVA_OPTIONS environment variable are picked up twice.
>
> If an option cannot be accepted twice (e.g., -agentlib), the application
> fails to s
On Thu, 8 Dec 2022 14:33:01 GMT, Maurizio Cimadamore
wrote:
>> This PR proposes changing variable names and text from "session" to "scope".
>> The proposed changes are only for the FFM API and not other parts like the
>> Vector API and test classes.
>
> src/java.base/share/classes/jdk/internal
On Thu, 8 Dec 2022 14:11:29 GMT, Per Minborg wrote:
> This PR proposes changing variable names and text from "session" to "scope".
> The proposed changes are only for the FFM API and not other parts like the
> Vector API and test classes.
src/java.base/share/classes/jdk/internal/foreign/Abstra
This PR proposes changing variable names and text from "session" to "scope".
The proposed changes are only for the FFM API and not other parts like the
Vector API and test classes.
-
Commit messages:
- Rename session to scope
Changes: https://git.openjdk.org/jdk/pull/11593/files
On Thu, 8 Dec 2022 08:17:44 GMT, Per Minborg wrote:
>> This PR suggests renaming various names from "session" to "scope" in
>> accordance with https://openjdk.org/jeps/434
>>
>> The PRs contains changes for several sub-components.
>
> Per Minborg has updated the pull request incrementally with
On Wed, 7 Dec 2022 21:55:43 GMT, Per Minborg wrote:
> This PR suggests renaming various names from "session" to "scope" in
> accordance with https://openjdk.org/jeps/434
>
> The PRs contains changes for several sub-components.
This pull request has been closed without being integrated.
--
Newer version of IntelliJ IDEA introduces new
[inspection](https://youtrack.jetbrains.com/issue/IDEA-301797/IDEA-should-report-redundant-array-length-check-in-certain-cases)
detecting redundant array length check in snippets like
void iterate(T[] items) {
if (items.length == 0) {
return;
On Thu, 8 Dec 2022 10:48:40 GMT, Jaikiran Pai wrote:
> Can I get a review of this change which fixes the copyright header on the
> test file?
This pull request has now been integrated.
Changeset: 2f426cd6
Author:Jaikiran Pai
URL:
https://git.openjdk.org/jdk/commit/2f426cd68b28c8bf5
On Thu, 8 Dec 2022 10:48:40 GMT, Jaikiran Pai wrote:
> Can I get a review of this change which fixes the copyright header on the
> test file?
Thank you Alan for the quick review. Local testing of this change shows that it
works. I'll go ahead with the integration.
-
PR: https://g
On Thu, 8 Dec 2022 10:48:40 GMT, Jaikiran Pai wrote:
> Can I get a review of this change which fixes the copyright header on the
> test file?
Marked as reviewed by alanb (Reviewer).
-
PR: https://git.openjdk.org/jdk/pull/11583
Can I get a review of this change which fixes the copyright header on the test
file?
-
Commit messages:
- 8298375: Bad copyright header in
test/jdk/java/lang/Character/Supplementary.java
Changes: https://git.openjdk.org/jdk/pull/11583/files
Webrev: https://webrevs.openjdk.org/?re
On Tue, 6 Dec 2022 18:39:28 GMT, Sergey Tsypanov wrote:
>> I found out that this code
>>
>> public class Main {
>> public static void main(String[] args) {
>> String s = "Hello world!";
>> char[] chars = s.toCharArray();
>> int point = Character.codePointAt(chars, -1,
On Fri, 2 Dec 2022 12:44:18 GMT, Sergey Tsypanov wrote:
> I found out that this code
>
> public class Main {
> public static void main(String[] args) {
> String s = "Hello world!";
> char[] chars = s.toCharArray();
> int point = Character.codePointAt(chars, -1, 1);
>
> This PR suggests renaming various names from "session" to "scope" in
> accordance with https://openjdk.org/jeps/434
>
> The PRs contains changes for several sub-components.
Per Minborg has updated the pull request incrementally with one additional
commit since the last revision:
Revert ren
33 matches
Mail list logo