Revert "Do not call setFullyDefined() in ReplicateStatementContext" 38/94838/2
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 27 Jan 2021 11:06:22 +0000 (12:06 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 27 Jan 2021 11:50:23 +0000 (12:50 +0100)
This reverts commit 5a24e6e2e40ce961f016ee37ab44f5a0a46ec514. As it
turns out this can actually fail.

Change-Id: I3733be139390f959ec451bde76d1ea1b9e1ddced
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
yang/yang-parser-reactor/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/ReplicaStatementContext.java

index 6e4daf1255fab50309bdf5fb85f3164de1a1b771..dc2d5dec0f7ed0af34c15824aeb6dd015cd541d8 100644 (file)
@@ -7,7 +7,6 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.reactor;
 
-import static com.google.common.base.Verify.verify;
 import static java.util.Objects.requireNonNull;
 
 import java.util.Collection;
@@ -41,8 +40,9 @@ final class ReplicaStatementContext<A, D extends DeclaredStatement<A>, E extends
         this.parent = requireNonNull(parent);
         this.source = requireNonNull(source);
         if (source.isSupportedToBuildEffective()) {
-            verify(source.fullyDefined(), "Source %s is not fully defined", source);
             source.incRef();
+            // FIXME: is this call really needed? it is inherited from source
+            setFullyDefined();
         }
     }