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

Andrey Turbanov duke at openjdk.java.net
Wed Oct 27 18:25:44 UTC 2021


On Tue, 26 Oct 2021 15:18:26 GMT, Weijun Wang <weijun at openjdk.org> 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/classes/sun/security/rsa/RSAPSSSignature.java line 211:
> 
>> 209:         AlgorithmParameterSpec keyParams = rsaKey.getParams();
>> 210:         // validate key parameters
>> 211:         if (!isCompatible(keyParams, this.sigParams)) {
> 
> Why not remove the assignment on line 209? This is more consistent with your fix this time.

Ok. Removed.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6092


More information about the security-dev mailing list