RFR: 8273484: Cleanup unnecessary null comparison before instanceof check in java.naming
Andrey Turbanov
github.com+741251+turbanoff at openjdk.java.net
Wed Sep 8 10:06:32 UTC 2021
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
-------------
Commit messages:
- [PATCH] Cleanup unnecessary null comparison before instanceof check in java.naming
Changes: https://git.openjdk.java.net/jdk/pull/5374/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5374&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8273484
Stats: 41 lines in 12 files changed: 0 ins; 11 del; 30 mod
Patch: https://git.openjdk.java.net/jdk/pull/5374.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/5374/head:pull/5374
PR: https://git.openjdk.java.net/jdk/pull/5374
More information about the core-libs-dev
mailing list