Re: RFR: 8267521: Post JEP 411 refactoring: maximum covering > 50K [v3]

2021-05-27 Thread Weijun Wang
On Fri, 28 May 2021 03:12:35 GMT, Phil Race wrote: >> There *is* a tiny refactoring here: a new variable `s2` is introduced so the >> 2nd `doPrivileged` call on line 636 is now also in a declaration statement >> (for `s2`) and therefore annotatable. Without this I cannot add the >> annotation

Re: RFR: 8267521: Post JEP 411 refactoring: maximum covering > 50K [v3]

2021-05-27 Thread Phil Race
On Fri, 28 May 2021 02:50:55 GMT, Weijun Wang wrote: >> src/java.desktop/share/classes/java/awt/Component.java line 630: >> >>> 628: } >>> 629: >>> 630: @SuppressWarnings("removal") >> >> I'm confused. I thought the reason this wasn't done in the JEP >> implementation PR is

Re: RFR: 8267521: Post JEP 411 refactoring: maximum covering > 50K [v3]

2021-05-27 Thread Weijun Wang
On Thu, 27 May 2021 17:42:56 GMT, Phil Race wrote: >> Weijun Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> update FtpClient.java > > src/java.desktop/share/classes/java/awt/Component.java line 630: > >> 628: } >> 629:

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v5]

2021-05-27 Thread Weijun Wang
On Thu, 27 May 2021 20:16:25 GMT, Weijun Wang wrote: >> Please review this implementation of [JEP >> 411](https://openjdk.java.net/jeps/411). >> >> The code change is divided into 3 commits. Please review them one by one. >> >> 1. >>

JEP 411: Missing use-case: user functions in an RDBMS

2021-05-27 Thread Chapman Flack
Hello, I see I am another person relatively late to stumble on this "well publicized" JEP. (I am not sure how to recommend the publicity could have been better handled, but apparently the avenues that were used aren't ones that reached me.) I maintain, on a volunteer basis, the extension for Java

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v5]

2021-05-27 Thread Weijun Wang
> Please review this implementation of [JEP > 411](https://openjdk.java.net/jeps/411). > > The code change is divided into 3 commits. Please review them one by one. > > 1. > https://github.com/openjdk/jdk/commit/576161d15423f58281e384174d28c9f9be7941a1 > The essential change for this JEP,

Re: RFR: 8267521: Post JEP 411 refactoring: maximum covering > 50K [v3]

2021-05-27 Thread Phil Race
On Fri, 21 May 2021 20:37:30 GMT, Weijun Wang wrote: >> The code change refactors classes that have a `SuppressWarnings("removal")` >> annotation that covers more than 50KB of code. The big annotation is often >> quite faraway from the actual deprecated API usage it is suppressing, and >>

Re: RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v4]

2021-05-27 Thread Phil Race
On Mon, 24 May 2021 13:53:34 GMT, Weijun Wang wrote: >> Please review this implementation of [JEP >> 411](https://openjdk.java.net/jeps/411). >> >> The code change is divided into 3 commits. Please review them one by one. >> >> 1. >>

Integrated: 8267123: Remove RMI Activation

2021-05-27 Thread Stuart Marks
On Tue, 25 May 2021 18:04:45 GMT, Stuart Marks wrote: > This is the implementation of [JEP 407](https://openjdk.java.net/jeps/407). > > This is a fairly straightforward removal of this component. > - Activation implementation classes removed > - Activation tests removed > - adjustments to

Re: RFR: 8267587: Update java.util to use enhanced switch [v6]

2021-05-27 Thread Daniel Fuchs
On Wed, 26 May 2021 02:22:42 GMT, Tagir F. Valeev wrote: >> Inspired by PR#4088. Most of the changes are done automatically using >> IntelliJ IDEA refactoring. Some manual adjustments are also performed, >> including indentations, moving comments, extracting common cast out of >> switch

Re: Fuzzing for java.security.* (and other libraries)

2021-05-27 Thread Fabian Meumertzheim
Hi Sean, On Thu, May 27, 2021 at 2:35 PM Sean Mullan wrote: > Hi Fabian, > > Thanks for posting this and your interest in helping to test and improve > the quality of the Java core libraries. One comment/request below: > > On 5/17/21 9:09 AM, Fabian Meumertzheim wrote: > > (Crosspost from

Re: Fuzzing for java.security.* (and other libraries)

2021-05-27 Thread Sean Mullan
Hi Fabian, Thanks for posting this and your interest in helping to test and improve the quality of the Java core libraries. One comment/request below: On 5/17/21 9:09 AM, Fabian Meumertzheim wrote: (Crosspost from core-libs-dev@:

Integrated: 8267817: [TEST] Remove unnecessary init in test/micro/org/openjdk/bench/javax/crypto/full/AESGCMBench:setup

2021-05-27 Thread Dongbo He
On Thu, 27 May 2021 02:11:59 GMT, Dongbo He wrote: > decryptCipher will be reinitialized in decrypt, which will loses all > previously-acquired state. Therefore, it's not necessary to initialize in > setup. This pull request has now been integrated. Changeset: 85f61652 Author:Dongbo He

Re: RFR: 8264774: Implementation of Foreign Function and Memory API (Incubator) [v26]

2021-05-27 Thread Maurizio Cimadamore
> This PR contains the API and implementation changes for JEP-412 [1]. A more > detailed description of such changes, to avoid repetitions during the review > process, is included as a separate comment. > > [1] - https://openjdk.java.net/jeps/412 Maurizio Cimadamore has updated the pull

Re: RFR: 8180571: Refactor sun/security/pkcs11 shell tests to plain java tests and fix failures

2021-05-27 Thread Fernando Guallini
On Thu, 27 May 2021 09:55:10 GMT, Sibabrata Sahoo wrote: >> Refactor the following shell tests to Java: >> - security/pkcs11/KeyStore/Basic.sh >> - security/pkcs11/KeyStore/ClientAuth.sh >> - security/pkcs11/KeyStore/SecretKeysBasic.sh >> - security/pkcs11/Provider/ConfigQuotedString.sh >> -

Re: RFR: 8264774: Implementation of Foreign Function and Memory API (Incubator) [v25]

2021-05-27 Thread Maurizio Cimadamore
> This PR contains the API and implementation changes for JEP-412 [1]. A more > detailed description of such changes, to avoid repetitions during the review > process, is included as a separate comment. > > [1] - https://openjdk.java.net/jeps/412 Maurizio Cimadamore has updated the pull

Re: RFR: 8267817: [TEST] Remove unnecessary init in test/micro/org/openjdk/bench/javax/crypto/full/AESGCMBench:setup

2021-05-27 Thread Claes Redestad
On Thu, 27 May 2021 02:11:59 GMT, Dongbo He wrote: > decryptCipher will be reinitialized in decrypt, which will loses all > previously-acquired state. Therefore, it's not necessary to initialize in > setup. Looks good. - Marked as reviewed by redestad (Reviewer). PR:

Re: RFR: 8180571: Refactor sun/security/pkcs11 shell tests to plain java tests and fix failures

2021-05-27 Thread Sibabrata Sahoo
On Tue, 18 May 2021 13:19:53 GMT, Fernando Guallini wrote: > Refactor the following shell tests to Java: > - security/pkcs11/KeyStore/Basic.sh > - security/pkcs11/KeyStore/ClientAuth.sh > - security/pkcs11/KeyStore/SecretKeysBasic.sh > - security/pkcs11/Provider/ConfigQuotedString.sh > -

RFR: 8179880: Refactor javax/security shell tests to plain java tests

2021-05-27 Thread Sibabrata Sahoo
This change converts JTREG shell Test scripts to Java equivalent. - Commit messages: - 8179880: Refactor javax/security shell tests to plain java tests Changes: https://git.openjdk.java.net/jdk/pull/4220/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=4220=00 Issue: