Fix implementation nullness guarantees 79/94079/1
authorRobert Varga <robert.varga@pantheon.tech>
Sun, 6 Dec 2020 16:16:39 +0000 (17:16 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Sun, 6 Dec 2020 16:20:22 +0000 (17:20 +0100)
We are promising @NonNull in the contract, make sure implementations
do the same.

Change-Id: Ib895ffedc48c221ab4a98a4a318101bc3df46bc4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
yang/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/ContainerSchemaNodes.java
yang/yang-data-util/src/main/java/org/opendaylight/yangtools/yang/data/util/OperationAsContainer.java

index 337928f7241ea38bb2b0a3485055e1d84178bfa9..8010b84fbb395a1b1b16c00a40a0d99012e54099 100644 (file)
@@ -16,6 +16,7 @@ import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Maps;
 import java.util.Collection;
 import java.util.Optional;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.ActionDefinition;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchemaNode;
@@ -104,7 +105,7 @@ public final class ContainerSchemaNodes {
         }
 
         @Override
-        public Collection<? extends MustDefinition> getMustConstraints() {
+        public Collection<? extends @NonNull MustDefinition> getMustConstraints() {
             return ImmutableList.of();
         }
 
index 0aa9b414d8843a5c7c17ea54f27b77f328731323..33b182c34ecd4f4d4b5bc476af7af6aafad73eea 100644 (file)
@@ -62,12 +62,12 @@ public class OperationAsContainer extends ForwardingObject implements ContainerL
     }
 
     @Override
-    public final Collection<? extends TypeDefinition<?>> getTypeDefinitions() {
+    public final Collection<? extends @NonNull TypeDefinition<?>> getTypeDefinitions() {
         return delegate.getTypeDefinitions();
     }
 
     @Override
-    public final Collection<? extends GroupingDefinition> getGroupings() {
+    public final Collection<? extends @NonNull GroupingDefinition> getGroupings() {
         return delegate.getGroupings();
     }
 
@@ -177,7 +177,7 @@ public class OperationAsContainer extends ForwardingObject implements ContainerL
     }
 
     @Override
-    public final Collection<? extends MustDefinition> getMustConstraints() {
+    public final Collection<? extends @NonNull MustDefinition> getMustConstraints() {
         return ImmutableSet.of();
     }