RFR: 8259223: Simplify boolean expression in the SunJSSE provider

2021-01-04 Thread Xue-Lei Andrew Fan
There are some boolean expressions that could be improved for better readability in the SunJSSE provider implementation. For example: - if (cert instanceof X509Certificate == false) { + if (!(cert instanceof X509Certificate)) { - return isDTLS ? true : (id >= TLS10.id); + return isDTLS || (id

Integrated: 8259069: Fields could be final

2021-01-04 Thread Xue-Lei Andrew Fan
On Mon, 4 Jan 2021 18:06:24 GMT, Xue-Lei Andrew Fan wrote: > There are a few fields in the SunJSSE provider implementation could be final. > > Code clean up to easy the maintenance, no new test. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8259069 This pull request has now been integrated

Re: RFR: 8258833: Cancel multi-part cipher operations in SunPKCS11 after failures

2021-01-04 Thread Valerie Peng
On Mon, 28 Dec 2020 16:24:43 GMT, Martin Balao wrote: > When a multi-part cipher operation fails in SunPKCS11 (i.e. because of an > invalid block size), we now cancel the operation before returning the > underlying Session to the Session Manager. This allows to use the returned > Session for a

Re: RFR: 8257733: Move module-specific data from make to respective module [v4]

2021-01-04 Thread Phil Race
On Tue, 15 Dec 2020 22:56:15 GMT, Magnus Ihse Bursie wrote: >> A lot (but not all) of the data in make/data is tied to a specific module. >> For instance, the publicsuffixlist is used by java.base, and fontconfig by >> java.desktop. (A few directories, like mainmanifest, is *actually* used by

Re: RFR: 8259069: Fields could be final

2021-01-04 Thread Bradford Wetmore
On Mon, 4 Jan 2021 18:06:24 GMT, Xue-Lei Andrew Fan wrote: > There are a few fields in the SunJSSE provider implementation could be final. > > Code clean up to easy the maintenance, no new test. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8259069 Marked as reviewed by wetmore (Reviewer).

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Hans Boehm
On Mon, Jan 4, 2021 at 8:34 AM Peter Levart wrote: > > On Mon, 4 Jan 2021 15:57:33 GMT, Richard Reingruber wrote: > > >> The bug title and the PR title need to be the same. > >> Editing either one is fine. > > > > But wouldn't it be legal for a compiler (java to bytecode or bytecode to > > machin

Re: RFR: 8248862: Implement Enhanced Pseudo-Random Number Generators [v9]

2021-01-04 Thread Jim Laskey
> This PR is to introduce a new random number API for the JDK. The primary API > is found in RandomGenerator and RandomGeneratorFactory. Further description > can be found in the JEP https://openjdk.java.net/jeps/356 . > > javadoc can be found at > http://cr.openjdk.java.net/~jlaskey/prng/doc/a

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Richard Reingruber
On Mon, 4 Jan 2021 14:27:09 GMT, Peter Levart wrote: >> See: https://bugs.openjdk.java.net/browse/JDK-8259021 >> See also discussion in thread: >> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html > > Peter Levart has updated the pull request incrementally with one

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Richard Reingruber
On Mon, 4 Jan 2021 17:45:25 GMT, Mandy Chung wrote: >> Peter Levart has updated the pull request incrementally with one additional >> commit since the last revision: >> >> revert the unrelated change > > Marked as reviewed by mchung (Reviewer). > > > _Mailing list message from [Hans Boehm]

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Mandy Chung
On Mon, 4 Jan 2021 14:27:09 GMT, Peter Levart wrote: >> See: https://bugs.openjdk.java.net/browse/JDK-8259021 >> See also discussion in thread: >> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html > > Peter Levart has updated the pull request incrementally with one

RFR: 8259069: Fields could be final

2021-01-04 Thread Xue-Lei Andrew Fan
There are a few fields in the SunJSSE provider implementation could be final. Code clean up to easy the maintenance, no new test. Bug: https://bugs.openjdk.java.net/browse/JDK-8259069 - Commit messages: - 8259069: Fields could be final Changes: https://git.openjdk.java.net/jdk/pul

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Aleksey Shipilev
On Mon, 4 Jan 2021 14:27:09 GMT, Peter Levart wrote: >> See: https://bugs.openjdk.java.net/browse/JDK-8259021 >> See also discussion in thread: >> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html > > Peter Levart has updated the pull request incrementally with one

Re: RFR: 8259021: SharedSecrets should avoid double racy reads from non-volatile fields [v2]

2021-01-04 Thread Peter Levart
On Mon, 4 Jan 2021 15:57:33 GMT, Richard Reingruber wrote: >> The bug title and the PR title need to be the same. >> Editing either one is fine. > > But wouldn't it be legal for a compiler (java to bytecode or bytecode to > machinecode) to replace references of my_local_copy with references to >

Re: RFR: 8259021 avoid double racy reads from non-volatile fields of SharedSecrets [v2]

2021-01-04 Thread Richard Reingruber
On Mon, 4 Jan 2021 15:11:27 GMT, Roger Riggs wrote: >> Peter Levart has updated the pull request incrementally with one additional >> commit since the last revision: >> >> revert the unrelated change > > The bug title and the PR title need to be the same. > Editing either one is fine. But wo

Re: RFR: 8257733: Move module-specific data from make to respective module [v4]

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 22:56:15 GMT, Magnus Ihse Bursie wrote: >> A lot (but not all) of the data in make/data is tied to a specific module. >> For instance, the publicsuffixlist is used by java.base, and fontconfig by >> java.desktop. (A few directories, like mainmanifest, is *actually* used by

Re: RFR: 8259021 avoid double racy reads from non-volatile fields of SharedSecrets [v2]

2021-01-04 Thread Roger Riggs
On Mon, 4 Jan 2021 14:27:09 GMT, Peter Levart wrote: >> See: https://bugs.openjdk.java.net/browse/JDK-8259021 >> See also discussion in thread: >> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html > > Peter Levart has updated the pull request incrementally with one

Re: RFR: 8259021 avoid double racy reads from non-volatile fields of SharedSecrets [v2]

2021-01-04 Thread Claes Redestad
On Mon, 4 Jan 2021 14:27:09 GMT, Peter Levart wrote: >> See: https://bugs.openjdk.java.net/browse/JDK-8259021 >> See also discussion in thread: >> https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html > > Peter Levart has updated the pull request incrementally with one

Re: RFR: 8259021 avoid double racy reads from non-volatile fields of SharedSecrets [v2]

2021-01-04 Thread Peter Levart
> See: https://bugs.openjdk.java.net/browse/JDK-8259021 > See also discussion in thread: > https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html Peter Levart has updated the pull request incrementally with one additional commit since the last revision: revert the unr

Re: RFR: 8248862: Implement Enhanced Pseudo-Random Number Generators

2021-01-04 Thread Jim Laskey
On Wed, 18 Nov 2020 13:45:12 GMT, Jim Laskey wrote: > This PR is to introduce a new random number API for the JDK. The primary API > is found in RandomGenerator and RandomGeneratorFactory. Further description > can be found in the JEP https://openjdk.java.net/jeps/356 . > > javadoc can be foun

Re: RFR: 8259021 avoid double racy reads from non-volatile fields of SharedSecrets

2021-01-04 Thread Aleksey Shipilev
On Thu, 31 Dec 2020 10:02:01 GMT, Peter Levart wrote: > See: https://bugs.openjdk.java.net/browse/JDK-8259021 > See also discussion in thread: > https://mail.openjdk.java.net/pipermail/core-libs-dev/2020-December/072798.html Looks good, but can we not do the behavioral change in `ensureClassIn