Use an instanceof expression 75/107675/1
authorRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Sep 2023 19:01:59 +0000 (21:01 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Sep 2023 19:01:59 +0000 (21:01 +0200)
We have an-if-and-a-cast, use an instanceof expression instead.

Change-Id: Iee350a79fe740e2434ea1ce35071aa63315f1ae6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
data/yang-data-tree-ri/src/main/java/org/opendaylight/yangtools/yang/data/tree/impl/AbstractValidation.java

index d7383f9d08ed6e8f591f3064f3612f87f7e1ca0e..de2488fb78bc510a4abfa8c83f1f4639f517a987 100644 (file)
@@ -83,13 +83,12 @@ abstract class AbstractValidation extends ModificationApplyOperation {
     final void checkApplicable(final ModificationPath path, final NodeModification modification,
             final Optional<? extends TreeNode> current, final Version version) throws DataValidationFailedException {
         delegate.checkApplicable(path, modification, current, version);
-        if (!(modification instanceof ModifiedNode)) {
+        if (!(modification instanceof ModifiedNode modified)) {
             // FIXME: 7.0.0: turn this into a verify?
             LOG.debug("Could not validate {}, does not implement expected class {}", modification, ModifiedNode.class);
             return;
         }
 
-        final ModifiedNode modified = (ModifiedNode) modification;
         if (delegate instanceof AbstractValidation) {
             checkApplicable(path, verifyNotNull(modified.getValidatedNode(delegate, current)));
             return;