Re: RFR: 8275918: Remove unused local variables in java.base security code [v3]

2022-01-10 Thread Weijun Wang
On Wed, 27 Oct 2021 18:49:26 GMT, Andrey Turbanov wrote: >> Cleanup unused local variables. Looks like they are leftovers after >> refactoring. > > Andrey Turbanov has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains three commits: > >

Re: RFR: 8275918: Remove unused local variables in java.base security code [v3]

2022-01-10 Thread Andrey Turbanov
On Wed, 27 Oct 2021 18:49:26 GMT, Andrey Turbanov wrote: >> Cleanup unused local variables. Looks like they are leftovers after >> refactoring. > > Andrey Turbanov has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains three commits: > >

Re: RFR: 8275918: Remove unused local variables in java.base security code [v3]

2021-10-27 Thread Andrey Turbanov
> Cleanup unused local variables. Looks like they are leftovers after > refactoring. Andrey Turbanov has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits: - Merge remote-tracking branch 'origin/master' into remove_unused_l

Re: RFR: 8275918: Remove unused local variables in java.base security code [v2]

2021-10-27 Thread Andrey Turbanov
> Cleanup unused local variables. Looks like they are leftovers after > refactoring. Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision: 8275918: Remove unused local variables in java.base security code remove local variable --

Re: RFR: 8275918: Remove unused local variables in java.base security code [v2]

2021-10-27 Thread Andrey Turbanov
On Tue, 26 Oct 2021 15:18:26 GMT, Weijun Wang wrote: >> Andrey Turbanov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8275918: Remove unused local variables in java.base security code >> remove local variable > > src/java.base/share/

Re: RFR: 8275918: Remove unused local variables in java.base security code

2021-10-26 Thread Weijun Wang
On Sat, 23 Oct 2021 14:04:07 GMT, Andrey Turbanov wrote: > Cleanup unused local variables. Looks like they are leftovers after > refactoring. src/java.base/share/classes/sun/security/rsa/RSAPSSSignature.java line 211: > 209: AlgorithmParameterSpec keyParams = rsaKey.getParams(); > 210:

RFR: 8275918: Remove unused local variables in java.base security code

2021-10-26 Thread Andrey Turbanov
Cleanup unused local variables. Looks like they are leftovers after refactoring. - Commit messages: - [PATCH] Remove unused local variables in java.base security code Changes: https://git.openjdk.java.net/jdk/pull/6092/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=609