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