BUG-9265: Switch empty type mapping from Void to Empty

[yangtools.git] / yang / yang-data-impl / src / main / java / org / opendaylight / yangtools / yang / data / impl / schema / nodes / AbstractImmutableNormalizedValueNode.java

diff --git a/yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/nodes/AbstractImmutableNormalizedValueNode.java b/yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/nodes/AbstractImmutableNormalizedValueNode.java
index a37a44450e5df1b6ff83bfa284d9d7e7361c0387..31f8bab90dcd668012ee1a980e0c114397491ac8 100644 (file)
--- a/yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/nodes/AbstractImmutableNormalizedValueNode.java
+++ b/yang/yang-data-impl/src/main/java/org/opendaylight/yangtools/yang/data/impl/schema/nodes/AbstractImmutableNormalizedValueNode.java
@@ -7,39 +7,28 @@
  */
 package org.opendaylight.yangtools.yang.data.impl.schema.nodes;
 
-import javax.annotation.Nullable;
+import static java.util.Objects.requireNonNull;
+
+import javax.annotation.Nonnull;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgument;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public abstract class AbstractImmutableNormalizedValueNode<K extends PathArgument, V> extends
         AbstractImmutableNormalizedNode<K, V> {
 
-    private static final Logger LOG = LoggerFactory.getLogger(AbstractImmutableNormalizedValueNode.class);
-    @Nullable
+    @Nonnull
     private final V value;
 
-    protected AbstractImmutableNormalizedValueNode(final K nodeIdentifier, @Nullable final V value) {
+    protected AbstractImmutableNormalizedValueNode(final K nodeIdentifier, @Nonnull final V value) {
         super(nodeIdentifier);
-
-        /*
-         * Null value is allowed for empty type definition so it should be debug,
-         * but still we are logging it in case we need to debug missing values.
-         */
-        // FIXME: one we do not map YANG 'void' to java.lang.Void we should be enforcing non-null here
-        if (value == null) {
-            LOG.debug("The value of node {} is null", nodeIdentifier.getNodeType());
-        }
-        this.value = value;
+        this.value = requireNonNull(value);
     }
 
-    @Nullable
     @Override
     public final V getValue() {
         return wrapValue(value);
     }
 
-    @Nullable
+    @Nonnull
     protected final V value() {
         return value;
     }