Migrate DocumentedNode/Status to JDT annotations 07/77207/1
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 23 Oct 2018 13:42:28 +0000 (15:42 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 23 Oct 2018 13:52:12 +0000 (15:52 +0200)
Switch DocumentedNode and its implementation to JDT nullable
annotations.

JIRA: YANGTOOLS-907
Change-Id: I7eea3429d1ee7617f0731c7416456a44e2caffbe
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
24 files changed:
yang/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/DocumentedNode.java
yang/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/Status.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/AbstractSchemaContext.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/AbstractBaseType.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/AbstractDerivedType.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/AbstractRestrictedType.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/AbstractTypeDefinition.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/BitImpl.java
yang/yang-model-util/src/main/java/org/opendaylight/yangtools/yang/model/util/type/EnumPairImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/AbstractEffectiveDocumentedNode.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/AbstractEffectiveModule.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/AbstractEffectiveSchemaNode.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/AbstractEffectiveSimpleDataNodeContainer.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/UnknownEffectiveStatementBase.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/augment/AugmentEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/bit/BitEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/choice/ImplicitCaseSchemaNode.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/deviation/DeviationEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/enum_/EnumEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/extension/ExtensionEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/grouping/GroupingEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/notification/NotificationEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/refine/RefineEffectiveStatementImpl.java
yang/yang-parser-rfc7950/src/main/java/org/opendaylight/yangtools/yang/parser/rfc7950/stmt/uses/UsesEffectiveStatementImpl.java

index b66e447587bd2f38d787bf15752d2af5b6d6f13a..d6c387ae28d993239da56fc6bbfbf07dcf0b4329 100644 (file)
@@ -10,7 +10,7 @@ package org.opendaylight.yangtools.yang.model.api;
 import com.google.common.collect.ImmutableList;
 import java.util.List;
 import java.util.Optional;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 
 /**
  * Node which can have documentation assigned.
@@ -35,7 +35,7 @@ public interface DocumentedNode {
      *
      * @return list of unknown schema nodes defined under this node.
      */
-    @Nonnull default List<UnknownSchemaNode> getUnknownSchemaNodes() {
+    default @NonNull List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return ImmutableList.of();
     }
 
@@ -46,6 +46,6 @@ public interface DocumentedNode {
          * @return status of this node which represents the argument of the YANG
          *         <code>status</code> substatement
          */
-        @Nonnull Status getStatus();
+        @NonNull Status getStatus();
     }
 }
index af91f69bc410257af775649731af3f91332acfb8..55f5ccdcf079648fb4a0c97bb0428817e67fe8d8 100644 (file)
@@ -7,28 +7,27 @@
  */
 package org.opendaylight.yangtools.yang.model.api;
 
+import org.eclipse.jdt.annotation.NonNullByDefault;
+
 /**
  * Enumeration describing YANG 'status' statement. If no status is specified, the
  * default is CURRENT.
  */
+@NonNullByDefault
 public enum Status {
-
     /**
      * CURRENT means that the definition is current and valid.
      */
     CURRENT,
-
     /**
      * DEPRECATED indicates an obsolete definition, but it permits new/
      * continued implementation in order to foster interoperability with
      * older/existing implementations.
      */
     DEPRECATED,
-
     /**
      * OBSOLETE means the definition is obsolete and SHOULD NOT be implemented
      * and/or can be removed from implementations.
      */
     OBSOLETE
-
 }
index 43b22a45c05dd8f221d3516f5042d07570f22a2f..15b42215da51b551f1a1e805acdb1915f09a31e8 100644 (file)
@@ -181,13 +181,11 @@ public abstract class AbstractSchemaContext implements SchemaContext {
         return SchemaPath.ROOT;
     }
 
-    @Nonnull
     @Override
     public Status getStatus() {
         return Status.CURRENT;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         final List<UnknownSchemaNode> result = new ArrayList<>();
index 7b205a877ec6b62c4a1acb3bd00a5476b6c12d0d..043ebcd8495dfe4e444698ea1d95af3cb8cbc257 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.yangtools.yang.model.util.type;
 import com.google.common.collect.ImmutableList;
 import java.util.List;
 import java.util.Optional;
-import javax.annotation.Nonnull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.Status;
@@ -51,7 +50,6 @@ abstract class AbstractBaseType<T extends TypeDefinition<T>> extends AbstractTyp
         return Optional.empty();
     }
 
-    @Nonnull
     @Override
     public final Status getStatus() {
         return Status.CURRENT;
index 99feb4780dbe7225d0bf46513617e761425fc46d..1425ed43195fed3f3f79998aa8db374fed06ea19 100644 (file)
@@ -7,30 +7,31 @@
  */
 package org.opendaylight.yangtools.yang.model.util.type;
 
+import static java.util.Objects.requireNonNull;
+
 import com.google.common.base.MoreObjects;
-import com.google.common.base.Preconditions;
 import java.util.Collection;
 import java.util.Optional;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 
 abstract class AbstractDerivedType<T extends TypeDefinition<T>> extends AbstractTypeDefinition<T> {
-    private final T baseType;
+    private final @NonNull T baseType;
     private final Object defaultValue;
     private final String description;
     private final String reference;
-    private final Status status;
+    private final @NonNull Status status;
     private final String units;
 
     AbstractDerivedType(final T baseType, final SchemaPath path, final Object defaultValue, final String description,
             final String reference, final Status status, final String units,
             final Collection<UnknownSchemaNode> unknownSchemNodes) {
         super(path, unknownSchemNodes);
-        this.baseType = Preconditions.checkNotNull(baseType);
-        this.status = Preconditions.checkNotNull(status);
+        this.baseType = requireNonNull(baseType);
+        this.status = requireNonNull(status);
         this.defaultValue = defaultValue;
         this.description = description;
         this.reference = reference;
@@ -64,7 +65,6 @@ abstract class AbstractDerivedType<T extends TypeDefinition<T>> extends Abstract
         return Optional.ofNullable(reference);
     }
 
-    @Nonnull
     @Override
     public final Status getStatus() {
         return status;
index 3d94825691ba94bb92166ff5d06aa3d0e7fb2c31..77262c2280ada4f0c45c5ee4e5b584ba3c382141 100644 (file)
@@ -7,22 +7,23 @@
  */
 package org.opendaylight.yangtools.yang.model.util.type;
 
-import com.google.common.base.Preconditions;
+import static java.util.Objects.requireNonNull;
+
 import java.util.Collection;
 import java.util.Optional;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 
 abstract class AbstractRestrictedType<T extends TypeDefinition<T>> extends AbstractTypeDefinition<T> {
-    private final T baseType;
+    private final @NonNull T baseType;
 
     AbstractRestrictedType(final T baseType, final SchemaPath path,
             final Collection<UnknownSchemaNode> unknownSchemaNodes) {
         super(path, unknownSchemaNodes);
-        this.baseType = Preconditions.checkNotNull(baseType);
+        this.baseType = requireNonNull(baseType);
     }
 
     @Override
@@ -50,7 +51,6 @@ abstract class AbstractRestrictedType<T extends TypeDefinition<T>> extends Abstr
         return baseType.getReference();
     }
 
-    @Nonnull
     @Override
     public final Status getStatus() {
         return baseType.getStatus();
index 3ba29c4a8af8f1b37bad910336c67fbe46ebaded..983f55319d1c1250a7420386e3906f0d74bc5083 100644 (file)
@@ -7,11 +7,13 @@
  */
 package org.opendaylight.yangtools.yang.model.util.type;
 
-import com.google.common.base.Preconditions;
+import static java.util.Objects.requireNonNull;
+
 import com.google.common.collect.ImmutableList;
 import java.util.Collection;
 import java.util.List;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.Immutable;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
@@ -19,11 +21,11 @@ import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 
 abstract class AbstractTypeDefinition<T extends TypeDefinition<T>> implements Immutable, TypeDefinition<T> {
-    private final List<UnknownSchemaNode> unknownSchemaNodes;
-    private final SchemaPath path;
+    private final @NonNull List<UnknownSchemaNode> unknownSchemaNodes;
+    private final @NonNull SchemaPath path;
 
     AbstractTypeDefinition(final SchemaPath path, final Collection<UnknownSchemaNode> unknownSchemaNodes) {
-        this.path = Preconditions.checkNotNull(path);
+        this.path = requireNonNull(path);
         this.unknownSchemaNodes = ImmutableList.copyOf(unknownSchemaNodes);
     }
 
@@ -39,7 +41,6 @@ abstract class AbstractTypeDefinition<T extends TypeDefinition<T>> implements Im
         return path;
     }
 
-    @Nonnull
     @Override
     public final List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownSchemaNodes;
index 579c69f827a094a6b51e3e98d714607c65a672bf..8ef077d3908c49ca6165618d28ea61f045da3199 100644 (file)
@@ -7,11 +7,14 @@
  */
 package org.opendaylight.yangtools.yang.model.util.type;
 
-import com.google.common.base.Preconditions;
+import static com.google.common.base.Preconditions.checkArgument;
+import static java.util.Objects.requireNonNull;
+
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.Immutable;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
@@ -20,23 +23,23 @@ import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.type.BitsTypeDefinition.Bit;
 
 final class BitImpl implements Bit, Immutable {
-    private final List<UnknownSchemaNode> unknownNodes;
-    private final SchemaPath schemaPath;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull SchemaPath schemaPath;
     private final String description;
     private final String reference;
-    private final Status status;
+    private final @NonNull Status status;
     private final long position;
 
     BitImpl(final SchemaPath schemaPath, final long position, final String description,
             final String reference, final Status status, final List<UnknownSchemaNode> unknownNodes) {
-        this.schemaPath = Preconditions.checkNotNull(schemaPath, "Schema Path should not be null");
+        this.schemaPath = requireNonNull(schemaPath, "Schema Path should not be null");
 
-        Preconditions.checkArgument(position >= 0L && position <= 4294967295L, "Invalid position %s", position);
+        checkArgument(position >= 0L && position <= 4294967295L, "Invalid position %s", position);
         this.position = position;
         this.description = description;
         this.reference = reference;
-        this.status = Preconditions.checkNotNull(status);
-        this.unknownNodes = Preconditions.checkNotNull(unknownNodes);
+        this.status = requireNonNull(status);
+        this.unknownNodes = requireNonNull(unknownNodes);
     }
 
     @Nonnull
@@ -61,13 +64,11 @@ final class BitImpl implements Bit, Immutable {
         return Optional.ofNullable(reference);
     }
 
-    @Nonnull
     @Override
     public Status getStatus() {
         return status;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
@@ -114,5 +115,4 @@ final class BitImpl implements Bit, Immutable {
     public String toString() {
         return Bit.class.getSimpleName() + "[name=" + getQName().getLocalName() + ", position=" + position + "]";
     }
-
 }
index a0fd036802cbbd4e533a384825d27d7c03a3776d..918ee783c6e7fd3b2084459f3e49b748acc42710 100644 (file)
@@ -7,36 +7,36 @@
  */
 package org.opendaylight.yangtools.yang.model.util.type;
 
+import static java.util.Objects.requireNonNull;
+
 import com.google.common.base.MoreObjects;
-import com.google.common.base.Preconditions;
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.Immutable;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.type.EnumTypeDefinition.EnumPair;
 
 final class EnumPairImpl implements EnumPair, Immutable {
-    private final List<UnknownSchemaNode> unknownSchemaNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownSchemaNodes;
     private final String description;
     private final String reference;
-    private final Status status;
-    private final String name;
+    private final @NonNull Status status;
+    private final @NonNull String name;
     private final int value;
 
     EnumPairImpl(final String name, final int value, final String description, final String reference,
             final Status status, final List<UnknownSchemaNode> unknownSchemaNodes) {
-        this.name = Preconditions.checkNotNull(name);
+        this.name = requireNonNull(name);
         this.value = value;
         this.description = description;
         this.reference = reference;
-        this.status = Preconditions.checkNotNull(status);
-        this.unknownSchemaNodes = Preconditions.checkNotNull(unknownSchemaNodes);
+        this.status = requireNonNull(status);
+        this.unknownSchemaNodes = requireNonNull(unknownSchemaNodes);
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownSchemaNodes;
@@ -52,7 +52,6 @@ final class EnumPairImpl implements EnumPair, Immutable {
         return Optional.ofNullable(reference);
     }
 
-    @Nonnull
     @Override
     public Status getStatus() {
         return status;
index 5750165967ed97693bba62673098850a1ec584f3..d99e7d87a3801e6fc16016d33e0ce25bcdadc2e2 100644 (file)
@@ -8,7 +8,7 @@
 package org.opendaylight.yangtools.yang.parser.rfc7950.stmt;
 
 import java.util.Optional;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.model.api.DocumentedNode;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.meta.DeclaredStatement;
@@ -22,7 +22,7 @@ public abstract class AbstractEffectiveDocumentedNode<A, D extends DeclaredState
 
     private final String description;
     private final String reference;
-    private final Status status;
+    private final @NonNull Status status;
 
     /**
      * Constructor.
@@ -47,7 +47,6 @@ public abstract class AbstractEffectiveDocumentedNode<A, D extends DeclaredState
         return Optional.ofNullable(reference);
     }
 
-    @Nonnull
     @Override
     public final Status getStatus() {
         return status;
index 22148d3024deccb295e99bb93df279cfba286c46..3b7dc1f8aa8f9de01a4b9a279704615bea88aea1 100644 (file)
@@ -27,7 +27,6 @@ import java.util.Map;
 import java.util.Optional;
 import java.util.Set;
 import java.util.stream.Collectors;
-import javax.annotation.Nonnull;
 import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.SemVer;
 import org.opendaylight.yangtools.openconfig.model.api.OpenConfigVersionEffectiveStatement;
@@ -83,7 +82,7 @@ public abstract class AbstractEffectiveModule<D extends DeclaredStatement<String
     private final Set<Deviation> deviations;
     private final List<ExtensionDefinition> extensionNodes;
     private final Set<IdentitySchemaNode> identities;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final Map<QName, DataSchemaNode> childNodes;
     private final Set<GroupingDefinition> groupings;
     private final Set<UsesNode> uses;
@@ -365,7 +364,6 @@ public abstract class AbstractEffectiveModule<D extends DeclaredStatement<String
         return identities;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index 11b441531e5f0d0820b00abd8b32a1a2e3fd22fa..a7a9c45e3fe233002f026988450e447175caf554 100644 (file)
@@ -9,7 +9,7 @@ package org.opendaylight.yangtools.yang.parser.rfc7950.stmt;
 
 import com.google.common.collect.ImmutableList;
 import java.util.List;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaNode;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
@@ -21,8 +21,8 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 public abstract class AbstractEffectiveSchemaNode<D extends DeclaredStatement<QName>> extends
         AbstractSchemaEffectiveDocumentedNode<QName, D> implements SchemaNode {
 
-    private final SchemaPath path;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull SchemaPath path;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
 
     protected AbstractEffectiveSchemaNode(final StmtContext<QName, D, ?> ctx) {
         super(ctx);
@@ -37,19 +37,16 @@ public abstract class AbstractEffectiveSchemaNode<D extends DeclaredStatement<QN
         this.unknownNodes = listBuilder.build();
     }
 
-    @Nonnull
     @Override
     public final QName getQName() {
         return path.getLastComponent();
     }
 
-    @Nonnull
     @Override
     public final SchemaPath getPath() {
         return path;
     }
 
-    @Nonnull
     @Override
     public final List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index d4eb34cca5559aa21392fe86908cb39a3ca672a1..35452c9c9864cb5749cc8db26bc493682c8b9ca4 100644 (file)
@@ -15,6 +15,7 @@ import java.util.List;
 import java.util.Optional;
 import java.util.Set;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.AugmentationTarget;
@@ -33,7 +34,7 @@ public abstract class AbstractEffectiveSimpleDataNodeContainer<D extends Declare
         AbstractEffectiveDocumentedDataNodeContainer<QName, D> implements AugmentationTarget, DataSchemaNode {
 
     private final Set<AugmentationSchemaNode> augmentations;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final RevisionAwareXPath whenCondition;
     private final SchemaPath path;
     private final boolean configuration;
@@ -110,7 +111,6 @@ public abstract class AbstractEffectiveSimpleDataNodeContainer<D extends Declare
         return augmentations;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index c38505adc770791f2eb6a66672b7317a5daea36d..638929e7759aebc6470ee4a1c87ba3adbf5c43e2 100644 (file)
@@ -13,6 +13,7 @@ import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableList.Builder;
 import java.util.List;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
@@ -31,7 +32,7 @@ public abstract class UnknownEffectiveStatementBase<A, D extends UnknownStatemen
     private final boolean addedByAugmentation;
 
     private final ExtensionDefinition extension;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final QName nodeType;
     private final String nodeParameter;
 
index cbb6a815cc78cc8c94e9ac21f94190c8b13ceeee..dc7d0d9b96a16d3b82da0e53e4d9fb5ab4f54bc6 100644 (file)
@@ -14,7 +14,7 @@ import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QNameModule;
 import org.opendaylight.yangtools.yang.common.Revision;
 import org.opendaylight.yangtools.yang.model.api.ActionDefinition;
@@ -41,7 +41,7 @@ final class AugmentEffectiveStatementImpl
     private final Revision revision;
     private final Set<ActionDefinition> actions;
     private final Set<NotificationDefinition> notifications;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final RevisionAwareXPath whenCondition;
     private final AugmentationSchemaNode copyOf;
 
@@ -92,7 +92,6 @@ final class AugmentEffectiveStatementImpl
         return Optional.ofNullable(whenCondition);
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index 943dfefd926d60fb120462cb7d06f0b81ba188a7..d6af45fe096fd1085e04e6c110ba418d8b2cf788 100644 (file)
@@ -11,7 +11,7 @@ import com.google.common.collect.ImmutableList;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
@@ -30,7 +30,7 @@ public final class BitEffectiveStatementImpl extends AbstractEffectiveDocumented
     private final QName qname;
     private final SchemaPath schemaPath;
     private final Long declaredPosition;
-    private final List<UnknownSchemaNode> unknownSchemaNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownSchemaNodes;
 
     BitEffectiveStatementImpl(final StmtContext<QName, BitStatement, ?> ctx) {
         super(ctx);
@@ -69,7 +69,6 @@ public final class BitEffectiveStatementImpl extends AbstractEffectiveDocumented
         return schemaPath;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownSchemaNodes;
index fc678900a2a93460a2e02b2ad8d1bae2b1318c54..9801fb8bf860270e34599515df62d5a8298903d0 100644 (file)
@@ -12,7 +12,6 @@ import static java.util.Objects.requireNonNull;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import java.util.Collection;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;
@@ -28,7 +27,6 @@ import org.opendaylight.yangtools.yang.model.api.SchemaNode;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.UsesNode;
 
 // FIXME: hide this somewhere
@@ -77,12 +75,6 @@ public final class ImplicitCaseSchemaNode implements CaseSchemaNode, DerivableSc
         return path;
     }
 
-    @Nonnull
-    @Override
-    public List<UnknownSchemaNode> getUnknownSchemaNodes() {
-        return ImmutableList.of();
-    }
-
     @Override
     public Optional<String> getDescription() {
         return caseShorthandNode.getDescription();
@@ -93,7 +85,6 @@ public final class ImplicitCaseSchemaNode implements CaseSchemaNode, DerivableSc
         return caseShorthandNode.getReference();
     }
 
-    @Nonnull
     @Override
     public Status getStatus() {
         return caseShorthandNode.getStatus();
index 1211aefc6283872dc90dc8bb545674ace2c99f82..0938f18ceb2685cd0adad4b549e96614f2442937 100644 (file)
@@ -12,6 +12,7 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.Immutable;
 import org.opendaylight.yangtools.yang.model.api.DeviateDefinition;
 import org.opendaylight.yangtools.yang.model.api.Deviation;
@@ -32,7 +33,7 @@ final class DeviationEffectiveStatementImpl
     private final SchemaPath targetPath;
     private final String description;
     private final String reference;
-    private final List<UnknownSchemaNode> unknownSchemaNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownSchemaNodes;
     private final List<DeviateDefinition> deviateDefinitions;
 
     DeviationEffectiveStatementImpl(final StmtContext<SchemaNodeIdentifier, DeviationStatement, ?> ctx) {
index d8264925ffcad5130911e522a26f8153f36aae6e..54e6b9f0a9370d51376cad0ee4f26ac4715caa3b 100644 (file)
@@ -10,7 +10,7 @@ package org.opendaylight.yangtools.yang.parser.rfc7950.stmt.enum_;
 import com.google.common.collect.ImmutableList;
 import java.util.ArrayList;
 import java.util.List;
-import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.EnumEffectiveStatement;
@@ -22,7 +22,7 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 // FIXME: hide this class
 public final class EnumEffectiveStatementImpl extends AbstractEffectiveDocumentedNode<String, EnumStatement>
         implements EnumEffectiveStatement {
-    private final List<UnknownSchemaNode> unknownSchemaNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownSchemaNodes;
     private final String name;
     private final Integer declaredValue;
 
@@ -54,7 +54,6 @@ public final class EnumEffectiveStatementImpl extends AbstractEffectiveDocumente
         return declaredValue;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownSchemaNodes;
index 36b29275a276e3e61b8e60c975b12b833dd80a62..5858dbfe201af877feae08aa19f50ef2dbdff514 100644 (file)
@@ -16,6 +16,7 @@ import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.util.RecursiveObjectLeaker;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
@@ -67,9 +68,9 @@ final class ExtensionEffectiveStatementImpl extends AbstractEffectiveDocumentedN
 
     private final QName qname;
     private final String argument;
-    private final SchemaPath schemaPath;
+    private final @NonNull SchemaPath schemaPath;
 
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final boolean yin;
 
     private ExtensionEffectiveStatementImpl(
@@ -140,13 +141,11 @@ final class ExtensionEffectiveStatementImpl extends AbstractEffectiveDocumentedN
         return qname;
     }
 
-    @Nonnull
     @Override
     public SchemaPath getPath() {
         return schemaPath;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index 92c378d0561adb38cd0747eeed7a7d81b95ff47d..07045b18ec7c99449159cb5975c970e06083a525 100644 (file)
@@ -14,6 +14,7 @@ import java.util.List;
 import java.util.Objects;
 import java.util.Set;
 import javax.annotation.Nonnull;
+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.GroupingDefinition;
@@ -35,7 +36,7 @@ final class GroupingEffectiveStatementImpl
     private final boolean addedByUses;
     private final Set<ActionDefinition> actions;
     private final Set<NotificationDefinition> notifications;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
 
     GroupingEffectiveStatementImpl(
             final StmtContext<QName, GroupingStatement, EffectiveStatement<QName, GroupingStatement>> ctx) {
index ac12fc5a1f15aaa8553c54bf879e83ec2c33ca27..4dcd4d4f963cb6cfcb050bc01e03fc4e4dcb3a21 100644 (file)
@@ -16,6 +16,7 @@ import java.util.List;
 import java.util.Objects;
 import java.util.Set;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.MustDefinition;
@@ -36,7 +37,7 @@ final class NotificationEffectiveStatementImpl
     private final QName qname;
     private final SchemaPath path;
     private final Set<AugmentationSchemaNode> augmentations;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final boolean augmenting;
     private final boolean addedByUses;
     private final Collection<MustDefinition> mustConstraints;
@@ -97,7 +98,6 @@ final class NotificationEffectiveStatementImpl
         return augmentations;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index 6655b5a210173444cc1e5f62188f00c12bd92c1e..ee6a5b81002f897fdb8e2eb9c3ed3c2d1972c1fb 100644 (file)
@@ -11,6 +11,7 @@ import com.google.common.collect.ImmutableList;
 import java.util.List;
 import java.util.stream.Collectors;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.SchemaNode;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
@@ -27,8 +28,8 @@ public final class RefineEffectiveStatementImpl
         implements RefineEffectiveStatement, SchemaNode {
 
     private final QName qname;
-    private final SchemaPath path;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull SchemaPath path;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final SchemaNode refineTargetNode;
 
     RefineEffectiveStatementImpl(final StmtContext<SchemaNodeIdentifier, RefineStatement, ?> ctx) {
@@ -60,7 +61,6 @@ public final class RefineEffectiveStatementImpl
         return path;
     }
 
-    @Nonnull
     @Override
     public List<UnknownSchemaNode> getUnknownSchemaNodes() {
         return unknownNodes;
index 8c783950c728ed04e936fcff256dc3fd6c6a083b..bb38cafd3e5bfaee8c9e4e5cc9e28648b543c865 100644 (file)
@@ -19,6 +19,7 @@ import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;
 import javax.annotation.Nonnull;
+import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.RevisionAwareXPath;
@@ -44,7 +45,7 @@ final class UsesEffectiveStatementImpl extends AbstractEffectiveDocumentedNode<Q
     private final boolean addedByUses;
     private final Map<SchemaPath, SchemaNode> refines;
     private final Set<AugmentationSchemaNode> augmentations;
-    private final List<UnknownSchemaNode> unknownNodes;
+    private final @NonNull List<UnknownSchemaNode> unknownNodes;
     private final RevisionAwareXPath whenCondition;
 
     UsesEffectiveStatementImpl(final StmtContext<QName, UsesStatement, EffectiveStatement<QName, UsesStatement>> ctx) {