RFR: 8265426: Update java.security to use instanceof pattern variable [v3]

[Patrick Concannon]

On Thu, 6 May 2021 13:41:04 GMT, Weijun Wang <weijun at openjdk.org> wrote:

>> Patrick Concannon has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8265426: Reverted parameter names; removed redundant parenthesis
>
> src/java.base/share/classes/java/security/cert/CertPath.java line 185:
> 
>> 183: 
>> 184:         return other instanceof CertPath that
>> 185:                 && this.type.equals(that.getType())
> 
> I know `type` should never be null but let's change as little as possible by using `that.getType().equals(this.type)`.

Ordering swapped around. See cbf2841

> src/java.base/share/classes/java/security/spec/EllipticCurve.java line 176:
> 
>> 174:                 && (field.equals(other.field)
>> 175:                 && a.equals(other.a)
>> 176:                 && b.equals(other.b));
> 
> I suppose there's no need to put the last 3 checks between "(" and ")".

Parenthesis removed as suggested. See cbf2841

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

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