Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-10 Thread Mark Powers
On Fri, 10 Jun 2022 21:44:16 GMT, Weijun Wang wrote: >> A privilegedGetProperty method? That would be the subject of a new bug I >> think. > > Yes. JDK-8288271 has been created - PR: https://git.openjdk.org/jdk/pull/8319

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-10 Thread Weijun Wang
On Fri, 10 Jun 2022 00:35:16 GMT, Mark Powers wrote: >> src/java.base/share/classes/java/security/SecureRandom.java line 905: >> >>> 903: private static final Pattern pattern = >>> 904: Pattern.compile( >>> 905:

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-10 Thread Mark Powers
On Wed, 8 Jun 2022 16:01:48 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-10 Thread Mark Powers
On Wed, 8 Jun 2022 16:06:12 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-10 Thread Mark Powers
On Wed, 8 Jun 2022 15:53:06 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-08 Thread Mark Powers
On Wed, 8 Jun 2022 16:12:36 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-08 Thread Mark Powers
On Wed, 8 Jun 2022 16:12:05 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-08 Thread Mark Powers
On Wed, 8 Jun 2022 15:50:38 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-08 Thread Mark Powers
On Wed, 8 Jun 2022 15:46:28 GMT, Weijun Wang wrote: >> Mark Powers has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 11 commits: >> >> - Merge >> - fourth iteration >> - Merge >> - Merge >> - third iteration >> - Merge >> -

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-08 Thread Weijun Wang
On Tue, 7 Jun 2022 15:37:02 GMT, Mark Powers wrote: >> https://bugs.openjdk.java.net/browse/JDK-8285263 Minor cleanup could be done >> in java.security >> >> JDK-8273046 is the umbrella bug for this bug. The changes were too large for >> a single code review, so it was decided to split into

Re: RFR: JDK-8285263 Minor cleanup could be done in java.security [v5]

2022-06-07 Thread Mark Powers
> https://bugs.openjdk.java.net/browse/JDK-8285263 Minor cleanup could be done > in java.security > > JDK-8273046 is the umbrella bug for this bug. The changes were too large for > a single code review, so it was decided to split into smaller chunks. This is > one such chunk: > >