On Mon, 28 Jun 2021 17:48:37 GMT, Brian Burkhalter wrote:
>> Modify `java.io.ByteArrayInputStream` methods `read(byte[])` and
>> `read(byte[],int,int)` to return zero per the `InputStream` specification
>> when the byte array actual or specified length is zero.
>
> Brian Burkhalter has updated
> Forwardport JDK 17 -> JDK 18
Jesper Wilhelmsson has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains 113 commits:
- Merge
- 8269615: Fix for 8263640 broke Windows build
Reviewed-by: iklam, dcubed
- 8269268: JDWP: Properly fix
On Wed, 30 Jun 2021 00:31:34 GMT, Jesper Wilhelmsson
wrote:
> Forwardport JDK 17 -> JDK 18
This pull request has now been integrated.
Changeset: ee526a2e
Author:Jesper Wilhelmsson
URL:
https://git.openjdk.java.net/jdk/commit/ee526a2ea840aedb97b23538f9d624acbccebc97
Stats: 127
Forwardport JDK 17 -> JDK 18
-
Commit messages:
- Merge
- 8269034: AccessControlException for SunPKCS11 daemon threads
- 8269529: javax/swing/reliability/HangDuringStaticInitialization.java fails
in Windows debug build
- 8269232: assert(!is_jweak(handle)) failed: wrong method
On Tue, 22 Jun 2021 13:26:41 GMT, Sean Coffey wrote:
> Sufficient permissions missing if this code was ever to run with
> SecurityManager.
>
> Cleanest approach appears to be use of InnocuousThread to create the
> cleaner/poller threads.
> Test case coverage extended to cover the
On Tue, 29 Jun 2021 19:57:24 GMT, Roger Riggs wrote:
>> If I switch to a "non-weak" set or map, then it seems I can safely use the
>> source string as the key. Will using the Class object as a key prevent them
>> from unloading?
>
> Using a synchronized WeakHashMap with the class as the key
On Tue, 29 Jun 2021 19:35:40 GMT, Weijun Wang wrote:
>> Using a HashSet could use the callerClass as the key and be a stable
>> reference for having given the message.
>> or use a ConcurrentHashMap>, boolean> and avoid any separate
>> synchronization that would be needed with a HashSet or
On Tue, 29 Jun 2021 19:23:26 GMT, Roger Riggs wrote:
>> src/java.base/share/classes/java/lang/System.java line 337:
>>
>>> 335: = Collections.synchronizedMap(new WeakHashMap<>());
>>> 336: }
>>> 337:
>>
>> I wonder about the use of a WeakHashMap here. That may work well when
On Fri, 25 Jun 2021 12:10:18 GMT, Masanori Yano wrote:
> Hi all,
>
> Could you please review the 8269373 bug fixes?
>
> These tests call java.lang.ProcessBuilder in direct, so not used jtreg
> command option. To run non-localized tests, -Duser.language=en and
> -Duser.country=US options
On Tue, 29 Jun 2021 18:47:23 GMT, Daniel Fuchs wrote:
>> Add a cache to record which sources have called `System::setSecurityManager`
>> and only print out warning lines once for each.
>
> src/java.base/share/classes/java/lang/System.java line 337:
>
>> 335: =
On Mon, 28 Jun 2021 21:09:43 GMT, Weijun Wang wrote:
> Add a cache to record which sources have called `System::setSecurityManager`
> and only print out warning lines once for each.
src/java.base/share/classes/java/lang/System.java line 337:
> 335: =
On Mon, 28 Jun 2021 09:13:31 GMT, Jaikiran Pai wrote:
>> Can I please get a review for this proposed fix for the issue reported in
>> https://bugs.openjdk.java.net/browse/JDK-8190753?
>>
>> The commit here checks for the size of the zip entry before trying to create
>> a
On Tue, 29 Jun 2021 17:36:30 GMT, Jim Laskey wrote:
>> The wording of the @implSpec referred to internal methods in the
>> description. The patch rewords the @implSpec to be more descriptive of the
>> algorithm than the methods used.
>
> Jim Laskey has updated the pull request incrementally
On Tue, 29 Jun 2021 00:07:41 GMT, Sean Coffey wrote:
>> Sufficient permissions missing if this code was ever to run with
>> SecurityManager.
>>
>> Cleanest approach appears to be use of InnocuousThread to create the
>> cleaner/poller threads.
>> Test case coverage extended to cover the
On Mon, 28 Jun 2021 19:07:09 GMT, Roger Riggs wrote:
>> Jim Laskey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Not averaging
>
> src/java.base/share/classes/java/util/random/RandomGenerator.java line 644:
>
>> 642: *
>> 643:
> The wording of the @implSpec referred to internal methods in the description.
> The patch rewords the @implSpec to be more descriptive of the algorithm than
> the methods used.
Jim Laskey has updated the pull request incrementally with two additional
commits since the last revision:
-
- Original Message -
> From: "Roger Riggs"
> To: "core-libs-dev"
> Sent: Mardi 29 Juin 2021 18:44:04
> Subject: Re: [jdk17] RFR: 8269096: Add java.util.Objects.newIdentity method
> [v5]
> On Tue, 29 Jun 2021 05:47:03 GMT, Tagir F. Valeev wrote:
>
>>> Roger Riggs has updated the pull
On Tue, 29 Jun 2021 05:47:03 GMT, Tagir F. Valeev wrote:
>> Roger Riggs has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add test synchronizing on return value of Objecst.newIdentity()
>
> Probably it would be better to have an inner
Add a cache to record which source has called `System::setSecurityManager` and
only print out warning lines once for each.
-
Commit messages:
- renaming variables, two callers in test
- one warning for each caller
Changes: https://git.openjdk.java.net/jdk17/pull/166/files
On Tue, 29 Jun 2021 04:39:28 GMT, Jonathan Gibbons wrote:
> Please review a trivial `noreg-doc` fix for some javadoc tags for
> `java.lang.Runtime` for JDK17
This pull request has now been integrated.
Changeset: 25f9f19a
Author:Jonathan Gibbons
URL:
On Wed, 23 Jun 2021 13:37:59 GMT, Matthias Baesken wrote:
>> Hello, please review this PR; it extend the OSContainer API in order to also
>> support the pids controller of cgroups.
>>
>> I noticed that unlike the other controllers "cpu", "cpuset", "cpuacct",
>> "memory" on some older Linux
On Tue, 1 Jun 2021 09:30:40 GMT, Jaroslav Tulach
wrote:
> There doesn't seem to be much support for the complete changes in #4245. To
> get at least something useful from that endeavor I have extracted the test
> for existing behavior of `-XX:+PreserveAllAnnotations` and I am offering it
>
On Tue, 1 Jun 2021 09:30:40 GMT, Jaroslav Tulach
wrote:
> There doesn't seem to be much support for the complete changes in #4245. To
> get at least something useful from that endeavor I have extracted the test
> for existing behavior of `-XX:+PreserveAllAnnotations` and I am offering it
>
On Tue, 1 Jun 2021 09:30:40 GMT, Jaroslav Tulach
wrote:
> There doesn't seem to be much support for the complete changes in #4245. To
> get at least something useful from that endeavor I have extracted the test
> for existing behavior of `-XX:+PreserveAllAnnotations` and I am offering it
>
A word about "compatibility" with other jdk tools...
jar is a specific tool. It was initially conceived to be user-friendly
with users that are already familiar with tar tool:
"The syntax for the jar command resembles the syntax for the tar command."
Mixing-in behaviors not consistent with
Hi,
On 29/03/2021 12:46, Jaikiran Pai wrote:
Given the support of -C/-dir is new and -P is there for backwards
compatibility then I see the choices are:
* Issue an error when -P and -C/-dir are specified together with -x
* Silently ignore the -P option
* Ignore -C/-dir and the behavior
26 matches
Mail list logo
