On Wed, 12 Jan 2022 07:29:04 GMT, John Jiang wrote:
>> In class `sun.security.ec.ECDHKeyAgreement`, the last `mod()` in the below
>> line looks redundant,
>>
>> BigInteger lhs = y.modPow(BigInteger.valueOf(2), p).mod(p);
>>
>> I think this tiny change just be a code cleanup, so no test for it
On Wed, 12 Jan 2022 07:29:04 GMT, John Jiang wrote:
>> In class `sun.security.ec.ECDHKeyAgreement`, the last `mod()` in the below
>> line looks redundant,
>>
>> BigInteger lhs = y.modPow(BigInteger.valueOf(2), p).mod(p);
>>
>> I think this tiny change just be a code cleanup, so no test for it
> In class `sun.security.ec.ECDHKeyAgreement.java`, the last `mod()` in the
> below line looks redundant,
>
> BigInteger lhs = y.modPow(BigInteger.valueOf(2), p).mod(p);
John Jiang has updated the pull request incrementally with one additional
commit since the last revision:
update copyright
In class `sun.security.ec.ECDHKeyAgreement.java`, the last `mod()` in the below
line looks redundant,
BigInteger lhs = y.modPow(BigInteger.valueOf(2), p).mod(p);
-
Commit messages:
- 8279903: Redundant modulo operation in ECDHKeyAgreement
Changes: https://git.openjdk.java.net/jdk/