Fix InterningLeafSetNodeBuilder check 64/86564/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 25 Dec 2019 11:23:09 +0000 (12:23 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 25 Dec 2019 11:25:07 +0000 (12:25 +0100)
The instanceof check is wrong, as pointed out by SpotBugs, as we
want to check for LeafListSchemaNode not LeafSetNode.

Change-Id: If538beca159c100621d5661dc8dced5c1051dfe9
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/InterningLeafSetNodeBuilder.java

index e55b1036a6427c6efd7139fe19a2fa8fa664e82c..bfa14598f1620abece68b0a1142c27043acd67af 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.yangtools.yang.data.impl.schema;
 import static java.util.Objects.requireNonNull;
 
 import org.opendaylight.yangtools.yang.data.api.schema.LeafSetEntryNode;
-import org.opendaylight.yangtools.yang.data.api.schema.LeafSetNode;
 import org.opendaylight.yangtools.yang.data.impl.schema.builder.api.ListNodeBuilder;
 import org.opendaylight.yangtools.yang.data.impl.schema.builder.impl.ImmutableLeafSetNodeBuilder;
 import org.opendaylight.yangtools.yang.data.util.LeafsetEntryInterner;
@@ -30,7 +29,8 @@ final class InterningLeafSetNodeBuilder<T> extends ImmutableLeafSetNodeBuilder<T
     }
 
     private static LeafsetEntryInterner getInterner(final DataSchemaNode schema) {
-        return schema instanceof LeafSetNode ? LeafsetEntryInterner.forSchema((LeafListSchemaNode) schema) : null;
+        return schema instanceof LeafListSchemaNode ? LeafsetEntryInterner.forSchema((LeafListSchemaNode) schema)
+                : null;
     }
 
     static <T> ListNodeBuilder<T, LeafSetEntryNode<T>> create(final DataSchemaNode schema) {