Remove Rfc6020AbnfRule 96/102896/3
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 26 Oct 2022 15:44:39 +0000 (17:44 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 26 Oct 2022 17:14:23 +0000 (19:14 +0200)
Remove the annotation and update all its users, so it does not confuse
anyone.

Change-Id: Ief29c36f380a676dd42935eaf6b980eab99924bc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/ArgumentStatement.java
model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/DataDefinitionStatement.java
model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/ModuleHeaderGroup.java
model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/Rfc6020AbnfRule.java [deleted file]
model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/TypeStatement.java

index 30219a8a9eaf28539116fb3c2a775c5fd48f0d0f..3e9f4275d1b8b4ba683c4e7082045161450ff822 100644 (file)
@@ -7,7 +7,6 @@
  */
 package org.opendaylight.yangtools.yang.model.api.stmt;
 
-import java.util.Optional;
 import org.eclipse.jdt.annotation.Nullable;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.YangStmtMapping;
@@ -17,7 +16,6 @@ import org.opendaylight.yangtools.yang.model.api.meta.StatementDefinition;
 /**
  * Declared representation of a {@code argument} statement.
  */
-@Rfc6020AbnfRule("argument-stmt")
 public interface ArgumentStatement extends DeclaredStatement<QName> {
     @Override
     default StatementDefinition statementDefinition() {
@@ -25,8 +23,8 @@ public interface ArgumentStatement extends DeclaredStatement<QName> {
     }
 
     default @Nullable YinElementStatement getYinElement() {
-        final Optional<YinElementStatement> opt = findFirstDeclaredSubstatement(YinElementStatement.class);
-        return opt.isPresent() ? opt.get() : null;
+        final var opt = findFirstDeclaredSubstatement(YinElementStatement.class);
+        return opt.isPresent() ? opt.orElseThrow() : null;
     }
 }
 
index 7384c77078d5e67edbea4a9c38e9efb7ba9cbe91..e8f014e0276c3363a05e5c834f42586a7adccb24 100644 (file)
@@ -14,10 +14,10 @@ import org.opendaylight.yangtools.yang.common.QName;
  * and anydata.
  *
  * <p>
- * Defined in: <a href="https://tools.ietf.org/html/rfc6020#section-3">RFC6020, Section 3</a>
+ * Defined in <a href="https://tools.ietf.org/html/rfc6020#section-3">RFC6020, Section 3</a>, as {@code data-def-stmt}
+ * ABNF rule.
  */
-@Rfc6020AbnfRule("data-def-stmt")
-public interface DataDefinitionStatement extends DocumentedDeclaredStatement.WithStatus<QName>,
-        WhenStatementAwareDeclaredStatement<QName> {
+public interface DataDefinitionStatement
+        extends DocumentedDeclaredStatement.WithStatus<QName>, WhenStatementAwareDeclaredStatement<QName> {
 
 }
index 681dea49ed9181632d10c019f458dcb1a48712cd..dfb8eaf087c5c60003b4ca984633babd34d31ea8 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.yangtools.yang.model.api.stmt;
 import org.eclipse.jdt.annotation.NonNull;
 import org.eclipse.jdt.annotation.Nullable;
 
-@Rfc6020AbnfRule("module-header-stmts")
 public interface ModuleHeaderGroup {
 
     @Nullable YangVersionStatement getYangVersion();
diff --git a/model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/Rfc6020AbnfRule.java b/model/yang-model-api/src/main/java/org/opendaylight/yangtools/yang/model/api/stmt/Rfc6020AbnfRule.java
deleted file mode 100644 (file)
index 29598b5..0000000
+++ /dev/null
@@ -1,36 +0,0 @@
-/*
- * Copyright (c) 2015 Cisco Systems, Inc. and others.  All rights reserved.
- *
- * This program and the accompanying materials are made available under the
- * terms of the Eclipse Public License v1.0 which accompanies this distribution,
- * and is available at http://www.eclipse.org/legal/epl-v10.html
- */
-package org.opendaylight.yangtools.yang.model.api.stmt;
-
-import java.lang.annotation.Documented;
-import java.lang.annotation.Retention;
-import java.lang.annotation.RetentionPolicy;
-
-/**
- * References ABNF rule defined in RFC6020 - YANG Specification.
- *
- * <p>
- * An interface / class annotated with this annotation
- * is Java representation of data represented by ABNF rule
- * provided as {@link #value()}. Java representation
- * does not need to be direct,
- * but must retain all information in some, publicly
- * accessible form for consumers.
- * </p>
- * <p>
- * Note that this annotation is used currently only for documentation
- * and does not affect any runtime behaviour.
- * </p>
- */
-@Deprecated(since = "9.0.2", forRemoval = true)
-@Documented
-@Retention(RetentionPolicy.SOURCE)
-@interface Rfc6020AbnfRule {
-
-    String[] value();
-}
index c7e7ecc3c4eb958ee0cce9f23965f0f8b549e660..6f05dc5a1b77b7dc3a9afa6365f5c3687667d3cc 100644 (file)
@@ -8,7 +8,6 @@
 package org.opendaylight.yangtools.yang.model.api.stmt;
 
 import java.util.Collection;
-import java.util.Optional;
 import org.eclipse.jdt.annotation.NonNull;
 import org.eclipse.jdt.annotation.Nullable;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -19,14 +18,12 @@ import org.opendaylight.yangtools.yang.model.api.meta.StatementDefinition;
 /**
  * Declared representation of a {@code type} statement.
  */
-@Rfc6020AbnfRule("type-stmt")
 public interface TypeStatement extends DeclaredStatement<QName> {
     @Override
     default StatementDefinition statementDefinition() {
         return YangStmtMapping.TYPE;
     }
 
-    @Rfc6020AbnfRule("numerical-restrictions")
     // FIXME: 7.0.0: this interface does not have an implementation
     interface NumericalRestrictions extends TypeStatement {
         default @Nullable RangeStatement getRange() {
@@ -34,24 +31,22 @@ public interface TypeStatement extends DeclaredStatement<QName> {
         }
     }
 
-    @Rfc6020AbnfRule("decimal64-specification")
     interface Decimal64Specification extends TypeStatement {
         default @NonNull FractionDigitsStatement getFractionDigits() {
-            return findFirstDeclaredSubstatement(FractionDigitsStatement.class).get();
+            return findFirstDeclaredSubstatement(FractionDigitsStatement.class).orElseThrow();
         }
 
         default @Nullable RangeStatement getRange() {
-            final Optional<RangeStatement> opt = findFirstDeclaredSubstatement(RangeStatement.class);
-            return opt.isPresent() ? opt.get() : null;
+            final var opt = findFirstDeclaredSubstatement(RangeStatement.class);
+            return opt.isPresent() ? opt.orElseThrow() : null;
         }
     }
 
-    @Rfc6020AbnfRule("string-restrictions")
     // FIXME: 7.0.0: this interface does not have an implementation
     interface StringRestrictions extends TypeStatement {
         default @Nullable LengthStatement getLength() {
-            final Optional<LengthStatement> opt = findFirstDeclaredSubstatement(LengthStatement.class);
-            return opt.isPresent() ? opt.get() : null;
+            final var opt = findFirstDeclaredSubstatement(LengthStatement.class);
+            return opt.isPresent() ? opt.orElseThrow() : null;
         }
 
         default @NonNull Collection<? extends PatternStatement> getPatterns() {
@@ -59,34 +54,28 @@ public interface TypeStatement extends DeclaredStatement<QName> {
         }
     }
 
-    @Rfc6020AbnfRule("enum-specification")
     interface EnumSpecification extends TypeStatement {
-
         default @NonNull Collection<? extends EnumStatement> getEnums() {
             return declaredSubstatements(EnumStatement.class);
         }
     }
 
-    @Rfc6020AbnfRule("leafref-specification")
     interface LeafrefSpecification extends TypeStatement {
         default @NonNull PathStatement getPath() {
-            return findFirstDeclaredSubstatement(PathStatement.class).get();
+            return findFirstDeclaredSubstatement(PathStatement.class).orElseThrow();
         }
 
         /**
-         * Return require-instance statement child, if present. For RFC6020 semantics, this method always returns
-         * null.
+         * Return require-instance statement child, if present. For RFC6020 semantics, this method always returns null.
          *
          * @return require-instance statement, if present.
          */
         default @Nullable RequireInstanceStatement getRequireInstance() {
-            final Optional<RequireInstanceStatement> opt =
-                    findFirstDeclaredSubstatement(RequireInstanceStatement.class);
-            return opt.isPresent() ? opt.get() : null;
+            final var opt = findFirstDeclaredSubstatement(RequireInstanceStatement.class);
+            return opt.isPresent() ? opt.orElseThrow() : null;
         }
     }
 
-    @Rfc6020AbnfRule("instanceidentifier-specification")
     interface InstanceIdentifierSpecification extends TypeStatement {
         /**
          * Return require-instance statement child, if present. For RFC6020 semantics, this method always returns
@@ -95,13 +84,11 @@ public interface TypeStatement extends DeclaredStatement<QName> {
          * @return require-instance statement, if present.
          */
         default @Nullable RequireInstanceStatement getRequireInstance() {
-            final Optional<RequireInstanceStatement> opt =
-                    findFirstDeclaredSubstatement(RequireInstanceStatement.class);
-            return opt.isPresent() ? opt.get() : null;
+            final var opt = findFirstDeclaredSubstatement(RequireInstanceStatement.class);
+            return opt.isPresent() ? opt.orElseThrow() : null;
         }
     }
 
-    @Rfc6020AbnfRule("identityref-specification")
     interface IdentityRefSpecification extends TypeStatement {
         /**
          * Returns the base statements.
@@ -114,21 +101,18 @@ public interface TypeStatement extends DeclaredStatement<QName> {
         }
     }
 
-    @Rfc6020AbnfRule("bits-specification")
     interface BitsSpecification extends TypeStatement {
         default @NonNull Collection<? extends BitStatement> getBits() {
             return declaredSubstatements(BitStatement.class);
         }
     }
 
-    @Rfc6020AbnfRule("union-specification")
     interface UnionSpecification extends TypeStatement {
         default @NonNull Collection<? extends TypeStatement> getTypes() {
             return declaredSubstatements(TypeStatement.class);
         }
     }
 
-    @Rfc6020AbnfRule("binary-specification")
     interface BinarySpecification extends TypeStatement {
         default @NonNull Collection<? extends LengthStatement> getLength() {
             return declaredSubstatements(LengthStatement.class);