RFR: 8273484: Cleanup unnecessary null comparison before instanceof check in java.naming [v2]
Aleksei Efimov
aefimov at openjdk.java.net
Thu Sep 9 21:09:06 UTC 2021
On Thu, 9 Sep 2021 13:24:27 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:
>> Update code checks both non-null and instance of a class in java.naming module classes.
>> The checks and explicit casts could also be replaced with pattern matching for the instanceof operator.
>> For example:
>> The following code:
>>
>> return (obj != null &&
>> obj instanceof CompoundName &&
>> impl.equals(((CompoundName)obj).impl));
>>
>>
>> Can be simplified to:
>>
>>
>> return (obj instanceof CompoundName other) &&
>> impl.equals(other.impl);
>>
>>
>> See similar cleanup in java.base - https://bugs.openjdk.java.net/browse/JDK-8258422
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
>
> 8273484: Cleanup unnecessary null comparison before instanceof check in java.naming
The latest version looks good to me.
Our CI system is also happy with this patch - no `java.naming` test failures observed.
-------------
Marked as reviewed by aefimov (Committer).
PR: https://git.openjdk.java.net/jdk/pull/5374
More information about the core-libs-dev
mailing list