Remove appendCopyHistory from public view 29/58529/2
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 8 Jun 2017 15:47:53 +0000 (17:47 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 8 Jun 2017 16:13:33 +0000 (18:13 +0200)
This is an internal detail of the copy process, hence it should not
be visible in StmtContext.Mutable.

Change-Id: I890af7f9f8273fc3960c4acd5340fa97f5c3c6c2
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/spi/meta/StmtContext.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/RootStatementContext.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/StatementContextBase.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/SubstatementContext.java

index d174b9754392e40239ff14bedc466acc2322fb76..3d338b73b3592d17f99121ec0fb3510a099eafa6 100644 (file)
@@ -221,10 +221,6 @@ public interface StmtContext<A, D extends DeclaredStatement<A>, E extends Effect
 
         void setIsSupportedToBuildEffective(boolean isSupportedToBuild);
 
-        void appendCopyHistory(CopyType typeOfCopy, CopyHistory toAppend);
-
-        void setOriginalCtx(StmtContext<?, ?, ?> originalCtx);
-
         void setOrder(int order);
 
         // FIXME: this seems to be unused, but looks useful.
index 7bb341e73c5dc633de57bb2f739d5b800316a736..526756bd83b0d7ba47f4464ffb9fcfa628b224f8 100644 (file)
@@ -71,7 +71,7 @@ public class RootStatementContext<A, D extends DeclaredStatement<A>, E extends E
         this.setRootIdentifier(identifier);
     }
 
-    RootStatementContext(final RootStatementContext<A, D, E> original, final QNameModule newQNameModule,
+    private RootStatementContext(final RootStatementContext<A, D, E> original, final QNameModule newQNameModule,
         final CopyType typeOfCopy) {
         super(original);
 
index 423a0cabb9ff5936f3c29c0147915f29c31a737e..86236bb099b8e5c118a17ef9feb8e7fbf03b9f00 100644 (file)
@@ -167,8 +167,7 @@ public abstract class StatementContextBase<A, D extends DeclaredStatement<A>, E
         return copyHistory;
     }
 
-    @Override
-    public void appendCopyHistory(final CopyType typeOfCopy, final CopyHistory toAppend) {
+    final void appendCopyHistory(final CopyType typeOfCopy, final CopyHistory toAppend) {
         copyHistory = copyHistory.append(typeOfCopy, toAppend);
     }
 
@@ -177,8 +176,7 @@ public abstract class StatementContextBase<A, D extends DeclaredStatement<A>, E
         return originalCtx;
     }
 
-    @Override
-    public void setOriginalCtx(final StmtContext<?, ?, ?> originalCtx) {
+    final void setOriginalCtx(final StmtContext<?, ?, ?> originalCtx) {
         this.originalCtx = originalCtx;
     }
 
index b736a23c2821c699618816a9042623e9c68ef4d5..198830eaf380220c0683245b37af6407aa86d0f4 100644 (file)
@@ -79,7 +79,7 @@ final class SubstatementContext<A, D extends DeclaredStatement<A>, E extends Eff
     }
 
     @SuppressWarnings("unchecked")
-    SubstatementContext(final SubstatementContext<A, D, E> original, final QNameModule newQNameModule,
+    private SubstatementContext(final SubstatementContext<A, D, E> original, final QNameModule newQNameModule,
             final StatementContextBase<?, ?, ?> newParent, final CopyType typeOfCopy) {
         super(original);
         this.parent = newParent;