On Tue, 26 Oct 2021 15:18:26 GMT, Weijun Wang <[email protected]> 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