Cleanup yang-parser-impl 37/26937/2
authorRobert Varga <rovarga@cisco.com>
Mon, 14 Sep 2015 10:42:01 +0000 (12:42 +0200)
committerGerrit Code Review <gerrit@opendaylight.org>
Wed, 16 Sep 2015 14:19:15 +0000 (14:19 +0000)
All around fixes:
- methods which can be made static
- Optional instance sharing
- Raw types

Change-Id: I04e2e7190cd4ffeda7d8a7fc2530f2c8e7bb27a8
Signed-off-by: Robert Varga <rovarga@cisco.com>
34 files changed:
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/BuildGlobalContext.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/NamespaceStorageSupport.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/reactor/SourceSpecificContext.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/AugmentUtils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/BelongsToStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/IdentityStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/ImportStatementDefinition.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/PatternStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/TypeStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/ChoiceEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/EffectiveStatementBase.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/ExtendedTypeEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/ModuleEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/SubmoduleEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/type/BinaryEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/type/Decimal64SpecificationEffectiveStatementImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/stmt/rfc6020/effective/type/StringEffectiveStatementImpl.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/parser/impl/Bug2291Test.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/effective/build/test/EffectiveBuildTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/effective/build/test/EffectiveModulesAndSubmodulesTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/effective/build/test/EffectiveUsesRefineAndConstraintsTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/ListKeysTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/SchemaContextTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/TypesResolutionTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/YangModelValidationSubModuleTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/YangModelValidationTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/retest/YangParserTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/ControllerStmtParserTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/ImportResolutionBasicTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/IncludeResolutionTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/ModuleSourceTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/MoreRevisionsTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/YangTypes2StmtTest.java
yang/yang-parser-impl/src/test/java/org/opendaylight/yangtools/yang/stmt/test/YangTypesStmtTest.java

index ef04c15a351309fdfbc2d013e724961c3af4535b..a7041f2f14040c117557dc1f7bd75c0e7696e564 100644 (file)
@@ -7,29 +7,24 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.reactor;
 
-import org.opendaylight.yangtools.yang.parser.spi.meta.ModelActionBuilder.Prerequisite;
-import org.opendaylight.yangtools.yang.parser.spi.validation.ValidationBundlesNamespace;
-
-import java.util.Map.Entry;
-import java.util.Collection;
-import org.opendaylight.yangtools.yang.parser.spi.validation.ValidationBundlesNamespace.ValidationBundleType;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
-import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import com.google.common.base.Preconditions;
 import com.google.common.base.Throwables;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Lists;
 import java.util.ArrayList;
+import java.util.Collection;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Objects;
 import java.util.Set;
 import javax.annotation.Nonnull;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.meta.DeclaredStatement;
+import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import org.opendaylight.yangtools.yang.model.api.meta.IdentifierNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ModelProcessingPhase;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour;
@@ -42,7 +37,10 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.StatementSupport;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StatementSupportBundle;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
 import org.opendaylight.yangtools.yang.parser.spi.source.StatementStreamSource;
+import org.opendaylight.yangtools.yang.parser.spi.validation.ValidationBundlesNamespace;
+import org.opendaylight.yangtools.yang.parser.spi.validation.ValidationBundlesNamespace.ValidationBundleType;
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.SourceSpecificContext.PhaseCompletionProgress;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 
 class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBehaviour.Registry {
 
@@ -63,12 +61,12 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
     private ModelProcessingPhase currentPhase = ModelProcessingPhase.INIT;
     private ModelProcessingPhase finishedPhase = ModelProcessingPhase.INIT;
 
-    public BuildGlobalContext(Map<ModelProcessingPhase, StatementSupportBundle> supports) {
+    public BuildGlobalContext(final Map<ModelProcessingPhase, StatementSupportBundle> supports) {
         super();
         this.supports = Preconditions.checkNotNull(supports, "BuildGlobalContext#supports cannot be null");
     }
 
-    public BuildGlobalContext(Map<ModelProcessingPhase, StatementSupportBundle> supports,  Map<ValidationBundleType,Collection<?>> supportedValidation) {
+    public BuildGlobalContext(final Map<ModelProcessingPhase, StatementSupportBundle> supports,  final Map<ValidationBundleType,Collection<?>> supportedValidation) {
         super();
         this.supports = Preconditions.checkNotNull(supports, "BuildGlobalContext#supports cannot be null");
 
@@ -78,11 +76,11 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
         }
     }
 
-    public StatementSupportBundle getSupportsForPhase(ModelProcessingPhase currentPhase) {
+    public StatementSupportBundle getSupportsForPhase(final ModelProcessingPhase currentPhase) {
         return supports.get(currentPhase);
     }
 
-    public void addSource(@Nonnull StatementStreamSource source) {
+    public void addSource(@Nonnull final StatementStreamSource source) {
         sources.add(new SourceSpecificContext(this,source));
     }
 
@@ -102,7 +100,7 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
     }
 
     @Override
-    public <K, V, N extends IdentifierNamespace<K, V>> NamespaceBehaviourWithListeners<K, V, N> getNamespaceBehaviour(Class<N> type) {
+    public <K, V, N extends IdentifierNamespace<K, V>> NamespaceBehaviourWithListeners<K, V, N> getNamespaceBehaviour(final Class<N> type) {
         NamespaceBehaviourWithListeners<?, ?, ?> potential = supportedNamespaces.get(type);
         if (potential == null) {
             NamespaceBehaviour<K, V, N> potentialRaw = supports.get(currentPhase).getNamespaceBehaviour(type);
@@ -123,7 +121,7 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
         throw new NamespaceNotAvailableException("Namespace " + type + " is not available in phase " + currentPhase);
     }
 
-    public StatementDefinitionContext<?, ?, ?> getStatementDefinition(QName name) {
+    public StatementDefinitionContext<?, ?, ?> getStatementDefinition(final QName name) {
         StatementDefinitionContext<?, ?, ?> potential = definitions.get(name);
         if(potential == null) {
             StatementSupport<?, ?, ?> potentialRaw = supports.get(currentPhase).getStatementDefinition(name);
@@ -181,7 +179,7 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
         return new EffectiveSchemaContext(rootStatements,rootEffectiveStatements);
     }
 
-    private void startPhase(ModelProcessingPhase phase) {
+    private void startPhase(final ModelProcessingPhase phase) {
         Preconditions.checkState(Objects.equals(finishedPhase, phase.getPreviousPhase()));
         for(SourceSpecificContext source : sources) {
             source.startPhase(phase);
@@ -196,8 +194,8 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
         }
     }
 
-    private SomeModifiersUnresolvedException addSourceExceptions(SomeModifiersUnresolvedException buildFailure,
-            List<SourceSpecificContext> sourcesToProgress) {
+    private SomeModifiersUnresolvedException addSourceExceptions(final SomeModifiersUnresolvedException buildFailure,
+            final List<SourceSpecificContext> sourcesToProgress) {
         for(SourceSpecificContext failedSource : sourcesToProgress) {
             SourceException sourceEx = failedSource.failModifiers(currentPhase);
             buildFailure.addSuppressed(sourceEx);
@@ -242,7 +240,7 @@ class BuildGlobalContext extends NamespaceStorageSupport implements NamespaceBeh
         }
     }
 
-    private  void endPhase(ModelProcessingPhase phase) {
+    private  void endPhase(final ModelProcessingPhase phase) {
         Preconditions.checkState(currentPhase == phase);
         finishedPhase = currentPhase;
     }
index 27914951ba9472b0f554cf9ceb7f873acb3235b2..0f9b948496fb372be3ca2e3e2852738e98b34fde 100644 (file)
@@ -7,19 +7,18 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.reactor;
 
+import java.util.HashMap;
+import java.util.Map;
 import java.util.Map.Entry;
-
 import java.util.Set;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
 import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
-import java.util.HashMap;
-import java.util.Map;
 import org.opendaylight.yangtools.yang.model.api.meta.IdentifierNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour.NamespaceStorageNode;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceNotAvailableException;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StatementNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
 
 abstract class NamespaceStorageSupport implements NamespaceStorageNode {
 
@@ -31,43 +30,43 @@ abstract class NamespaceStorageSupport implements NamespaceStorageNode {
 
     public abstract NamespaceBehaviour.Registry getBehaviourRegistry();
 
-    protected void checkLocalNamespaceAllowed(Class<? extends IdentifierNamespace<?, ?>> type) {
+    protected void checkLocalNamespaceAllowed(final Class<? extends IdentifierNamespace<?, ?>> type) {
         // NOOP
     }
 
-    protected <K, V, N extends IdentifierNamespace<K, V>> void onNamespaceElementAdded(Class<N> type, K key, V value) {
+    protected <K, V, N extends IdentifierNamespace<K, V>> void onNamespaceElementAdded(final Class<N> type, final K key, final V value) {
         // NOOP
     }
 
     //<K,V,N extends IdentifierNamespace<K, V>> V
     //public final <K, VT, V extends VT ,N extends IdentifierNamespace<K, V>> VT getFromNamespace(Class<N> type, K key)
-    public final <K,V, KT extends K, N extends IdentifierNamespace<K, V>> V getFromNamespace(Class<N> type, KT key)
+    public final <K,V, KT extends K, N extends IdentifierNamespace<K, V>> V getFromNamespace(final Class<N> type, final KT key)
             throws NamespaceNotAvailableException {
         return getBehaviourRegistry().getNamespaceBehaviour(type).getFrom(this,key);
     }
 
-    public final <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromNamespace(Class<N> type){
+    public final <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromNamespace(final Class<N> type){
         return getBehaviourRegistry().getNamespaceBehaviour(type).getAllFrom(this);
     }
 
-    public final <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromCurrentStmtCtxNamespace(Class<N> type){
+    public final <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromCurrentStmtCtxNamespace(final Class<N> type){
         return (Map<K, V>) namespaces.get(type);
     }
 
-    public final <K,V, KT extends K, VT extends V,N extends IdentifierNamespace<K, V>> void addToNs(Class<N> type, KT key, VT value)
+    public final <K,V, KT extends K, VT extends V,N extends IdentifierNamespace<K, V>> void addToNs(final Class<N> type, final KT key, final VT value)
             throws NamespaceNotAvailableException {
         getBehaviourRegistry().getNamespaceBehaviour(type).addTo(this,key,value);
     }
 
     @SuppressWarnings({ "unchecked", "rawtypes" })
-    public final <K, N extends StatementNamespace<K, ?,?>> void addContextToNamespace(Class<N> type, K key, StmtContext<?, ?, ?> value)
+    public final <K, N extends StatementNamespace<K, ?,?>> void addContextToNamespace(final Class<N> type, final K key, final StmtContext<?, ?, ?> value)
             throws NamespaceNotAvailableException {
         getBehaviourRegistry().getNamespaceBehaviour((Class)type).addTo(this, key, value);
     }
 
     @SuppressWarnings("unchecked")
     @Override
-    public <K, V, N extends IdentifierNamespace<K, V>> V getFromLocalStorage(Class<N> type, K key) {
+    public <K, V, N extends IdentifierNamespace<K, V>> V getFromLocalStorage(final Class<N> type, final K key) {
         Map<K, V> localNamespace = (Map<K,V>) namespaces.get(type);
 
         V potential = null;
@@ -82,17 +81,17 @@ abstract class NamespaceStorageSupport implements NamespaceStorageNode {
         return potential;
     }
 
-    private <K, V, N extends IdentifierNamespace<K, V>> V getRegardlessOfRevision(ModuleIdentifier key,
-            Map<ModuleIdentifier, V> localNamespace) {
+    private static <K, V, N extends IdentifierNamespace<K, V>> V getRegardlessOfRevision(final ModuleIdentifier key,
+            final Map<ModuleIdentifier, V> localNamespace) {
 
-        if(localNamespace == null) {
+        if (localNamespace == null) {
             return null;
         }
 
         Set<Entry<ModuleIdentifier, V>> entrySet = localNamespace.entrySet();
         for (Entry<ModuleIdentifier, V> entry : entrySet) {
             ModuleIdentifier moduleIdentifierInMap = entry.getKey();
-            if(moduleIdentifierInMap.getName().equals(key.getName())) {
+            if (moduleIdentifierInMap.getName().equals(key.getName())) {
                 return entry.getValue();
             }
         }
@@ -101,14 +100,14 @@ abstract class NamespaceStorageSupport implements NamespaceStorageNode {
     }
 
     @Override
-    public <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromLocalStorage(Class<N> type) {
+    public <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAllFromLocalStorage(final Class<N> type) {
         @SuppressWarnings("unchecked")
         Map<K, V> localNamespace = (Map<K, V>) namespaces.get(type);
         return localNamespace;
     }
 
     @Override
-    public <K, V, N extends IdentifierNamespace<K, V>> void addToLocalStorage(Class<N> type, K key, V value) {
+    public <K, V, N extends IdentifierNamespace<K, V>> void addToLocalStorage(final Class<N> type, final K key, final V value) {
         @SuppressWarnings("unchecked")
         Map<K, V> localNamespace = (Map<K,V>) namespaces.get(type);
         if(localNamespace == null) {
index 7a5f7be94d85c7ed322163be584539e171831af6..a8c823e2ab8df964bf73b704990db5707f4e8b64 100644 (file)
@@ -7,13 +7,17 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.reactor;
 
-import javax.annotation.Nonnull;
-import org.opendaylight.yangtools.yang.model.api.stmt.ExtensionStatement;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.HashMultimap;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Multimap;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Objects;
+import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
 import org.opendaylight.yangtools.concepts.Mutable;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
@@ -22,6 +26,7 @@ import org.opendaylight.yangtools.yang.model.api.meta.DeclaredStatement;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import org.opendaylight.yangtools.yang.model.api.meta.IdentifierNamespace;
 import org.opendaylight.yangtools.yang.model.api.meta.StatementDefinition;
+import org.opendaylight.yangtools.yang.model.api.stmt.ExtensionStatement;
 import org.opendaylight.yangtools.yang.parser.spi.ExtensionNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ImportedNamespaceContext;
 import org.opendaylight.yangtools.yang.parser.spi.meta.InferenceException;
@@ -31,6 +36,7 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour.NamespaceStorageNode;
 import org.opendaylight.yangtools.yang.parser.spi.meta.NamespaceBehaviour.StorageNodeType;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StatementSupport;
+import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import org.opendaylight.yangtools.yang.parser.spi.source.PrefixToModule;
 import org.opendaylight.yangtools.yang.parser.spi.source.PrefixToModuleMap;
 import org.opendaylight.yangtools.yang.parser.spi.source.QNameToStatementDefinition;
@@ -41,7 +47,6 @@ import org.opendaylight.yangtools.yang.parser.spi.source.StatementStreamSource;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.BitsSpecificationImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Decimal64SpecificationImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.EnumSpecificationImpl;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.UnknownEffectiveStatementImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.IdentityRefSpecificationImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.InstanceIdentifierSpecificationImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.LeafrefSpecificationImpl;
@@ -49,12 +54,7 @@ import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.TypeUtils;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.UnionSpecificationImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.UnknownStatementImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
-import javax.annotation.Nullable;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.Objects;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.UnknownEffectiveStatementImpl;
 
 public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBehaviour.Registry, Mutable {
 
@@ -73,11 +73,11 @@ public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBeh
 
     private ModelProcessingPhase inProgressPhase;
     private ModelProcessingPhase finishedPhase = ModelProcessingPhase.INIT;
-    private QNameToStatementDefinitionMap qNameToStmtDefMap = new QNameToStatementDefinitionMap();
-    private PrefixToModuleMap prefixToModuleMap = new PrefixToModuleMap();
+    private final QNameToStatementDefinitionMap qNameToStmtDefMap = new QNameToStatementDefinitionMap();
+    private final PrefixToModuleMap prefixToModuleMap = new PrefixToModuleMap();
 
 
-    SourceSpecificContext(BuildGlobalContext currentContext, StatementStreamSource source) {
+    SourceSpecificContext(final BuildGlobalContext currentContext, final StatementStreamSource source) {
         this.source = source;
         this.currentContext = currentContext;
     }
@@ -86,11 +86,11 @@ public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBeh
         return inProgressPhase;
     }
 
-    StatementDefinitionContext<?, ?, ?> getDefinition(QName name) {
+    StatementDefinitionContext<?, ?, ?> getDefinition(final QName name) {
         return currentContext.getStatementDefinition(name);
     }
 
-    ContextBuilder<?, ?, ?> createDeclaredChild(StatementContextBase<?, ?, ?> current, QName name, StatementSourceReference ref) {
+    ContextBuilder<?, ?, ?> createDeclaredChild(final StatementContextBase<?, ?, ?> current, final QName name, final StatementSourceReference ref) {
         StatementDefinitionContext<?, ?, ?> def = getDefinition(name);
 
         if (def == null) {
@@ -101,8 +101,8 @@ public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBeh
                     final StatementContextBase<?,?,?> extension = (StatementContextBase<?, ?, ?>) currentContext
                             .getAllFromNamespace(ExtensionNamespace.class).get(key);
                     if (extension != null) {
-                        final QName qName = QName.create(((QName) ((SubstatementContext) extension).getStatementArgument())
-                                .getModule().getNamespace(), ((QName) ((SubstatementContext) extension).
+                        final QName qName = QName.create(((QName) ((SubstatementContext<?, ?, ?>) extension).getStatementArgument())
+                                .getModule().getNamespace(), ((QName) ((SubstatementContext<?, ?, ?>) extension).
                                 getStatementArgument()).getModule().getRevision(), extension.getIdentifier().getArgument());
 
                         def = new StatementDefinitionContext<>(new UnknownStatementImpl.Definition
@@ -159,7 +159,7 @@ public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBeh
     }
 
     @SuppressWarnings({"rawtypes", "unchecked"})
-    private ContextBuilder<?, ?, ?> createDeclaredRoot(StatementDefinitionContext<?, ?, ?> def, StatementSourceReference ref) {
+    private ContextBuilder<?, ?, ?> createDeclaredRoot(final StatementDefinitionContext<?, ?, ?> def, final StatementSourceReference ref) {
         return new ContextBuilder(def, ref) {
 
             @Override
@@ -330,7 +330,7 @@ public class SourceSpecificContext implements NamespaceStorageNode, NamespaceBeh
         }
     }
 
-    private StatementDefinitionContext<?, ?, ?> resolveTypeBodyStmts(String typeArgument) {
+    private static StatementDefinitionContext<?, ?, ?> resolveTypeBodyStmts(final String typeArgument) {
         switch (typeArgument) {
             case TypeUtils.DECIMAL64:
                 return new StatementDefinitionContext<>(new Decimal64SpecificationImpl.Definition());
index 9076c6873b3f07644a65a86f4c9e005679a5b891..984a862ad179014884602b44d8d1416718a25f5a 100644 (file)
@@ -51,7 +51,7 @@ public final class AugmentUtils {
     private AugmentUtils() {
     }
 
-    public static Iterable<QName> parseAugmentPath(StmtContext<?, ?, ?> ctx, String path) {
+    public static Iterable<QName> parseAugmentPath(final StmtContext<?, ?, ?> ctx, final String path) {
 
         if (path.matches(REGEX_PATH_REL1) || path.matches(REGEX_PATH_REL2)) {
             throw new IllegalArgumentException(
@@ -61,18 +61,18 @@ public final class AugmentUtils {
         return Utils.parseXPath(ctx, path);
     }
 
-    public static void copyFromSourceToTarget(StatementContextBase<?, ?, ?> sourceCtx,
-            StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
+    public static void copyFromSourceToTarget(final StatementContextBase<?, ?, ?> sourceCtx,
+            final StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
 
         copyDeclaredStmts(sourceCtx, targetCtx);
         copyEffectiveStmts(sourceCtx, targetCtx);
     }
 
-    public static void copyDeclaredStmts(StatementContextBase<?, ?, ?> sourceCtx,
-            StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
+    public static void copyDeclaredStmts(final StatementContextBase<?, ?, ?> sourceCtx,
+            final StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
 
         Collection<? extends StatementContextBase<?, ?, ?>> declaredSubStatements = sourceCtx.declaredSubstatements();
-        final List<StatementContextBase> subStatements = new Builder<StatementContextBase>()
+        final List<StatementContextBase<?, ?, ?>> subStatements = new Builder<StatementContextBase<?, ?, ?>>()
                 .addAll(targetCtx.declaredSubstatements()).addAll(targetCtx.effectiveSubstatements()).build();
         boolean sourceAndTargetInSameModule = Utils.getRootModuleQName(sourceCtx).equals(
                 Utils.getRootModuleQName(targetCtx));
@@ -92,11 +92,11 @@ public final class AugmentUtils {
         }
     }
 
-    public static void copyEffectiveStmts(StatementContextBase<?, ?, ?> sourceCtx,
-            StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
+    public static void copyEffectiveStmts(final StatementContextBase<?, ?, ?> sourceCtx,
+            final StatementContextBase<?, ?, ?> targetCtx) throws SourceException {
 
         Collection<? extends StatementContextBase<?, ?, ?>> effectiveSubstatements = sourceCtx.effectiveSubstatements();
-        final List<StatementContextBase> subStatements = new Builder<StatementContextBase>()
+        final List<StatementContextBase<?, ?, ?>> subStatements = new Builder<StatementContextBase<?, ?, ?>>()
                 .addAll(targetCtx.declaredSubstatements()).addAll(targetCtx.effectiveSubstatements()).build();
         boolean sourceAndTargetInSameModule = Utils.getRootModuleQName(sourceCtx).equals(
                 Utils.getRootModuleQName(targetCtx));
@@ -117,17 +117,17 @@ public final class AugmentUtils {
     }
 
     private static void validateNodeCanBeCopiedByAugment(final StatementContextBase<?, ?, ?> sourceCtx,
-            final List<StatementContextBase> targetSubStatements, boolean sourceAndTargetInSameModule) {
+            final List<StatementContextBase<?, ?, ?>> targetSubStatements, final boolean sourceAndTargetInSameModule) {
 
         if (sourceCtx.getPublicDefinition().getDeclaredRepresentationClass().equals(WhenStatement.class)) {
             return;
         }
 
         if (!sourceAndTargetInSameModule) {
-            final List<StatementContextBase> sourceSubStatements = new Builder<StatementContextBase>()
+            final List<StatementContextBase<?, ?, ?>> sourceSubStatements = new Builder<StatementContextBase<?, ?, ?>>()
                     .addAll(sourceCtx.declaredSubstatements()).addAll(sourceCtx.effectiveSubstatements()).build();
 
-            for (final StatementContextBase sourceSubStatement : sourceSubStatements) {
+            for (final StatementContextBase<?, ?, ?> sourceSubStatement : sourceSubStatements) {
                 if (sourceSubStatement.getPublicDefinition().getDeclaredRepresentationClass()
                         .equals(MandatoryStatement.class)) {
                     throw new IllegalArgumentException(
@@ -138,7 +138,7 @@ public final class AugmentUtils {
             }
         }
 
-        for (final StatementContextBase subStatement : targetSubStatements) {
+        for (final StatementContextBase<?, ?, ?> subStatement : targetSubStatements) {
 
             final boolean sourceIsDataNode = DataDefinitionStatement.class.isAssignableFrom(sourceCtx
                     .getPublicDefinition().getDeclaredRepresentationClass());
@@ -155,8 +155,8 @@ public final class AugmentUtils {
         }
     }
 
-    public static QNameModule getNewQNameModule(StatementContextBase<?, ?, ?> targetCtx,
-            StatementContextBase<?, ?, ?> sourceCtx) {
+    public static QNameModule getNewQNameModule(final StatementContextBase<?, ?, ?> targetCtx,
+            final StatementContextBase<?, ?, ?> sourceCtx) {
         Object targetStmtArgument = targetCtx.getStatementArgument();
 
         final StatementContextBase<?, ?, ?> root = sourceCtx.getRoot();
@@ -176,7 +176,7 @@ public final class AugmentUtils {
         }
     }
 
-    public static boolean needToCopyByAugment(StmtContext<?, ?, ?> stmtContext) {
+    public static boolean needToCopyByAugment(final StmtContext<?, ?, ?> stmtContext) {
 
         Set<StatementDefinition> noCopyDefSet = new HashSet<>();
         noCopyDefSet.add(Rfc6020Mapping.USES);
@@ -185,7 +185,7 @@ public final class AugmentUtils {
         return !noCopyDefSet.contains(def);
     }
 
-    public static boolean isReusedByAugment(StmtContext<?, ?, ?> stmtContext) {
+    public static boolean isReusedByAugment(final StmtContext<?, ?, ?> stmtContext) {
 
         Set<StatementDefinition> reusedDefSet = new HashSet<>();
         reusedDefSet.add(Rfc6020Mapping.TYPEDEF);
@@ -230,15 +230,16 @@ public final class AugmentUtils {
         StatementContextBase<?, ?, ?> augmentTargetCtx = null;
         for (final StatementContextBase<?, ?, ?> rootStatementCtx : rootStatementCtxList) {
             augmentTargetCtx = findCtxOfNodeInRoot(rootStatementCtx, augmentTargetNode);
-            if (augmentTargetCtx != null)
+            if (augmentTargetCtx != null) {
                 break;
+            }
         }
 
         return augmentTargetCtx;
     }
 
     @Nullable
-    public static StatementContextBase<?, ?, ?> findCtxOfNodeInSubstatements(StatementContextBase<?, ?, ?> rootStmtCtx,
+    public static StatementContextBase<?, ?, ?> findCtxOfNodeInSubstatements(final StatementContextBase<?, ?, ?> rootStmtCtx,
             final Iterable<QName> path) {
 
         StatementContextBase<?, ?, ?> parent = rootStmtCtx;
@@ -261,8 +262,8 @@ public final class AugmentUtils {
         return null;
     }
 
-    public static StatementContextBase<?, ?, ?> getSubstatementByQName(StatementContextBase<?, ?, ?> parent,
-            QName nextPathQName) {
+    public static StatementContextBase<?, ?, ?> getSubstatementByQName(final StatementContextBase<?, ?, ?> parent,
+            final QName nextPathQName) {
 
         Collection<StatementContextBase<?, ?, ?>> declaredSubstatement = parent.declaredSubstatements();
         Collection<StatementContextBase<?, ?, ?>> effectiveSubstatement = parent.effectiveSubstatements();
@@ -297,7 +298,7 @@ public final class AugmentUtils {
         return null;
     }
 
-    public static boolean isSupportedAugmentTarget(StatementContextBase<?, ?, ?> substatementCtx) {
+    public static boolean isSupportedAugmentTarget(final StatementContextBase<?, ?, ?> substatementCtx) {
 
         /*
          * :TODO Substatement must be allowed augment target type e.g. Container, etc... and must be not for example
@@ -315,7 +316,7 @@ public final class AugmentUtils {
     }
 
     @Nullable
-    public static StatementContextBase<?, ?, ?> findCtxOfNodeInRoot(StatementContextBase<?, ?, ?> rootStmtCtx,
+    public static StatementContextBase<?, ?, ?> findCtxOfNodeInRoot(final StatementContextBase<?, ?, ?> rootStmtCtx,
             final SchemaNodeIdentifier node) {
         return findCtxOfNodeInSubstatements(rootStmtCtx, node.getPathFromRoot());
     }
index 72065ac8bd732bfc660c9c99c66a1a5fb213197d..ddc3d81bbf112fac72e18f6270ca8be621761c5a 100644 (file)
@@ -33,7 +33,7 @@ public class BelongsToStatementImpl extends AbstractDeclaredStatement<String>
         implements BelongsToStatement {
 
     protected BelongsToStatementImpl(
-            StmtContext<String, BelongsToStatement, ?> context) {
+            final StmtContext<String, BelongsToStatement, ?> context) {
         super(context);
     }
 
@@ -46,19 +46,19 @@ public class BelongsToStatementImpl extends AbstractDeclaredStatement<String>
         }
 
         @Override
-        public String parseArgumentValue(StmtContext<?, ?, ?> ctx, String value) {
+        public String parseArgumentValue(final StmtContext<?, ?, ?> ctx, final String value) {
             return value;
         }
 
         @Override
         public BelongsToStatement createDeclared(
-                StmtContext<String, BelongsToStatement, ?> ctx) {
+                final StmtContext<String, BelongsToStatement, ?> ctx) {
             return new BelongsToStatementImpl(ctx);
         }
 
         @Override
         public EffectiveStatement<String, BelongsToStatement> createEffective(
-                StmtContext<String, BelongsToStatement, EffectiveStatement<String, BelongsToStatement>> ctx) {
+                final StmtContext<String, BelongsToStatement, EffectiveStatement<String, BelongsToStatement>> ctx) {
             return new BelongsEffectiveToStatementImpl(ctx);
         }
 
@@ -80,7 +80,7 @@ public class BelongsToStatementImpl extends AbstractDeclaredStatement<String>
                 }
 
                 @Override
-                public void prerequisiteFailed(Collection<? extends ModelActionBuilder.Prerequisite<?>> failed) throws InferenceException {
+                public void prerequisiteFailed(final Collection<? extends ModelActionBuilder.Prerequisite<?>> failed) throws InferenceException {
                     if (failed.contains(belongsToPrereq)) {
                         throw new InferenceException("Module from belongs-to was not found: " + belongsToCtx.getStatementArgument(), belongsToCtx
                                 .getStatementSourceReference());
@@ -89,7 +89,7 @@ public class BelongsToStatementImpl extends AbstractDeclaredStatement<String>
             });
         }
 
-        private ModuleIdentifier getModuleIdentifier(StmtContext.Mutable<String, BelongsToStatement, EffectiveStatement<String, BelongsToStatement>> belongsToCtx) {
+        private static ModuleIdentifier getModuleIdentifier(final StmtContext.Mutable<String, BelongsToStatement, EffectiveStatement<String, BelongsToStatement>> belongsToCtx) {
             String moduleName = belongsToCtx.getStatementArgument();
             return new ModuleIdentifierImpl(moduleName, Optional.<URI> absent(), Optional.of(SimpleDateFormatUtil.DEFAULT_BELONGS_TO_DATE));
         }
index ada4c730958c2aa93e5ab42498c479d41eaea486..798e59374b35e5f7162b5791b0625a742047d097 100644 (file)
@@ -7,28 +7,26 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020;
 
-import org.opendaylight.yangtools.antlrv4.code.gen.YangStatementParser;
-import org.opendaylight.yangtools.yang.parser.spi.IdentityNamespace;
-import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
-import org.opendaylight.yangtools.yang.parser.stmt.reactor.StatementContextBase;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.IdentityEffectiveStatementImpl;
-import org.opendaylight.yangtools.yang.model.api.stmt.BaseStatement;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.Rfc6020Mapping;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
-import org.opendaylight.yangtools.yang.model.api.stmt.IdentityStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.BaseStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.DescriptionStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.IdentityStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.ReferenceStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.StatusStatement;
+import org.opendaylight.yangtools.yang.parser.spi.IdentityNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.AbstractDeclaredStatement;
 import org.opendaylight.yangtools.yang.parser.spi.meta.AbstractStatementSupport;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
+import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.IdentityEffectiveStatementImpl;
 
 public class IdentityStatementImpl extends AbstractDeclaredStatement<QName>
         implements IdentityStatement {
 
     protected IdentityStatementImpl(
-            StmtContext<QName, IdentityStatement, ?> context) {
+            final StmtContext<QName, IdentityStatement, ?> context) {
         super(context);
     }
 
@@ -39,19 +37,19 @@ public class IdentityStatementImpl extends AbstractDeclaredStatement<QName>
         }
 
         @Override
-        public QName parseArgumentValue(StmtContext<?, ?, ?> ctx, String value) {
+        public QName parseArgumentValue(final StmtContext<?, ?, ?> ctx, final String value) {
             return Utils.qNameFromArgument(ctx, value);
         }
 
         @Override
         public IdentityStatement createDeclared(
-                StmtContext<QName, IdentityStatement, ?> ctx) {
+                final StmtContext<QName, IdentityStatement, ?> ctx) {
             return new IdentityStatementImpl(ctx);
         }
 
         @Override
         public EffectiveStatement<QName, IdentityStatement> createEffective(
-                StmtContext<QName, IdentityStatement, EffectiveStatement<QName, IdentityStatement>> ctx) {
+                final StmtContext<QName, IdentityStatement, EffectiveStatement<QName, IdentityStatement>> ctx) {
             return new IdentityEffectiveStatementImpl(ctx);
         }
 
index 8d3c0cc8ac5ca07f9426107c81fbc371c7537328..dd73f6c484f5c000bd733e89b28b9945ab6fd3e2 100644 (file)
@@ -9,21 +9,19 @@ package org.opendaylight.yangtools.yang.parser.stmt.rfc6020;
 
 import static org.opendaylight.yangtools.yang.parser.spi.meta.ModelProcessingPhase.SOURCE_LINKAGE;
 import static org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils.firstAttributeOf;
-
-import java.util.Set;
-
-import java.util.Map.Entry;
-import java.util.Map;
-import org.opendaylight.yangtools.yang.model.api.stmt.ModuleStatement;
 import com.google.common.base.Optional;
 import java.net.URI;
 import java.util.Collection;
 import java.util.Date;
+import java.util.Map;
+import java.util.Map.Entry;
+import java.util.Set;
 import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
 import org.opendaylight.yangtools.yang.model.api.Rfc6020Mapping;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.ImportStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.ModuleStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.PrefixStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.RevisionDateStatement;
 import org.opendaylight.yangtools.yang.parser.builder.impl.ModuleIdentifierImpl;
@@ -48,19 +46,19 @@ public class ImportStatementDefinition
     }
 
     @Override
-    public String parseArgumentValue(StmtContext<?, ?, ?> ctx, String value) {
+    public String parseArgumentValue(final StmtContext<?, ?, ?> ctx, final String value) {
         return value;
     }
 
     @Override
     public ImportStatement createDeclared(
-            StmtContext<String, ImportStatement, ?> ctx) {
+            final StmtContext<String, ImportStatement, ?> ctx) {
         return new ImportStatementImpl(ctx);
     }
 
     @Override
     public EffectiveStatement<String, ImportStatement> createEffective(
-            StmtContext<String, ImportStatement, EffectiveStatement<String, ImportStatement>> ctx) {
+            final StmtContext<String, ImportStatement, EffectiveStatement<String, ImportStatement>> ctx) {
         return new ImportEffectiveStatementImpl(ctx);
     }
 
@@ -107,8 +105,8 @@ public class ImportStatementDefinition
             }
 
             private Entry<ModuleIdentifier, StmtContext<?, ModuleStatement, EffectiveStatement<String, ModuleStatement>>> findRecentModule(
-                    ModuleIdentifier impIdentifier,
-                    Map<ModuleIdentifier, StmtContext<?, ModuleStatement, EffectiveStatement<String, ModuleStatement>>> allModules) {
+                    final ModuleIdentifier impIdentifier,
+                    final Map<ModuleIdentifier, StmtContext<?, ModuleStatement, EffectiveStatement<String, ModuleStatement>>> allModules) {
 
                 ModuleIdentifier recentModuleIdentifier = impIdentifier;
                 Entry<ModuleIdentifier, StmtContext<?, ModuleStatement, EffectiveStatement<String, ModuleStatement>>> recentModuleEntry = null;
@@ -134,7 +132,7 @@ public class ImportStatementDefinition
 
             @Override
             public void prerequisiteFailed(
-                    Collection<? extends Prerequisite<?>> failed)
+                    final Collection<? extends Prerequisite<?>> failed)
                     throws InferenceException {
                 if (failed.contains(imported)) {
                     throw new InferenceException(String.format(
@@ -145,8 +143,8 @@ public class ImportStatementDefinition
         });
     }
 
-    private ModuleIdentifier getImportedModuleIdentifier(
-            Mutable<String, ImportStatement, ?> stmt) throws SourceException {
+    private static ModuleIdentifier getImportedModuleIdentifier(
+            final Mutable<String, ImportStatement, ?> stmt) throws SourceException {
 
         String moduleName = stmt.getStatementArgument();
         Date revision = firstAttributeOf(stmt.declaredSubstatements(),
index 0d35ec52d63711453d0600a3a592f13c71476fee..f587a0df94ec43fb4c365138b3a0e554c74f7ccc 100644 (file)
@@ -23,8 +23,12 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.AbstractStatementSupport;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.PatternConstraintEffectiveImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.PatternEffectiveStatementImpl;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class PatternStatementImpl extends AbstractDeclaredStatement<PatternConstraint> implements PatternStatement {
+    private static final Logger LOG = LoggerFactory.getLogger(PatternStatementImpl.class);
+    private static final Optional<String> OPTIONAL_EMPTY = Optional.of("");
 
     protected PatternStatementImpl(StmtContext<PatternConstraint, PatternStatement, ?> context) {
         super(context);
@@ -40,17 +44,13 @@ public class PatternStatementImpl extends AbstractDeclaredStatement<PatternConst
 
         @Override
         public PatternConstraint parseArgumentValue(StmtContext<?, ?, ?> ctx, String value) {
-
-            final StringBuilder wrapPatternBuilder = new StringBuilder(value.length() + 2);
-            wrapPatternBuilder.append('^');
-            wrapPatternBuilder.append(value);
-            wrapPatternBuilder.append('$');
-            final String pattern = wrapPatternBuilder.toString();
+            final String pattern = "^" + value + '$';
 
             try {
                 Pattern.compile(pattern);
-                return new PatternConstraintEffectiveImpl(pattern, Optional.of(""), Optional.of(""));
+                return new PatternConstraintEffectiveImpl(pattern, OPTIONAL_EMPTY, OPTIONAL_EMPTY);
             } catch (PatternSyntaxException e) {
+                LOG.debug("Pattern {} failed to compile", pattern, e);
                 return null;
             }
         }
index e3a4cf58db00022389b3f4e6da3b2bffef26ec7c..aed2fda2786192031f08d97831c3c2463f6bb124 100644 (file)
@@ -7,11 +7,7 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020;
 
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.BinaryEffectiveStatementImpl;
-
 import java.util.Collection;
-import org.opendaylight.yangtools.yang.parser.stmt.reactor.StatementContextBase;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.ExtendedTypeEffectiveStatementImpl;
 import javax.annotation.Nonnull;
 import org.opendaylight.yangtools.yang.model.api.Rfc6020Mapping;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
@@ -20,6 +16,9 @@ import org.opendaylight.yangtools.yang.parser.spi.meta.AbstractDeclaredStatement
 import org.opendaylight.yangtools.yang.parser.spi.meta.AbstractStatementSupport;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
+import org.opendaylight.yangtools.yang.parser.stmt.reactor.StatementContextBase;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.ExtendedTypeEffectiveStatementImpl;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.BinaryEffectiveStatementImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.BooleanEffectiveStatementImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.EmptyEffectiveStatementImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.Int16EffectiveStatementImpl;
@@ -35,7 +34,7 @@ import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type.UInt8E
 public class TypeStatementImpl extends AbstractDeclaredStatement<String>
         implements TypeStatement {
 
-    protected TypeStatementImpl(StmtContext<String, TypeStatement, ?> context) {
+    protected TypeStatementImpl(final StmtContext<String, TypeStatement, ?> context) {
         super(context);
     }
 
@@ -48,29 +47,29 @@ public class TypeStatementImpl extends AbstractDeclaredStatement<String>
         }
 
         @Override
-        public String parseArgumentValue(StmtContext<?, ?, ?> ctx, String value)
+        public String parseArgumentValue(final StmtContext<?, ?, ?> ctx, final String value)
                 throws SourceException {
             return value;
         }
 
         @Override
         public TypeStatement createDeclared(
-                StmtContext<String, TypeStatement, ?> ctx) {
+                final StmtContext<String, TypeStatement, ?> ctx) {
             return new TypeStatementImpl(ctx);
         }
 
         @Override
         public EffectiveStatement<String, TypeStatement> createEffective(
-                StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx) {
+                final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx) {
 
             // :FIXME improve the test of isExtended - e.g. unknown statements,
             // etc..
             Collection<StatementContextBase<?, ?, ?>> declaredSubstatements = ctx
                     .declaredSubstatements();
-            boolean isExtended = declaredSubstatements.isEmpty() ? false
-                    : true;
-            if (isExtended)
+            boolean isExtended = !declaredSubstatements.isEmpty();
+            if (isExtended) {
                 return new ExtendedTypeEffectiveStatementImpl(ctx, true);
+            }
 
             switch (ctx.getStatementArgument()) {
             case TypeUtils.INT8:
index 6b3f848ec7f61951b8f96d22e0fe3d2d1dce61a3..2eedd1e02bff463bee0984f90b3b7fd0c13b3029 100644 (file)
@@ -7,36 +7,36 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective;
 
-import java.util.SortedSet;
-import java.util.TreeSet;
-import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.AnyXmlSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.LeafListSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
-import org.opendaylight.yangtools.yang.parser.builder.util.Comparators;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.TypeOfCopy;
-import java.util.Collection;
-import java.util.HashSet;
-import java.util.LinkedList;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
-import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
-import org.opendaylight.yangtools.yang.model.api.stmt.ChoiceStatement;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import com.google.common.base.Optional;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
+import java.util.Collection;
+import java.util.HashSet;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Set;
+import java.util.SortedSet;
+import java.util.TreeSet;
 import org.opendaylight.yangtools.yang.common.QName;
+import org.opendaylight.yangtools.yang.model.api.AnyXmlSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchema;
 import org.opendaylight.yangtools.yang.model.api.ChoiceCaseNode;
 import org.opendaylight.yangtools.yang.model.api.ChoiceSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ConstraintDefinition;
+import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.DerivableSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.LeafListSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 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.ChoiceStatement;
+import org.opendaylight.yangtools.yang.parser.builder.util.Comparators;
+import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
+import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.TypeOfCopy;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
 
 public class ChoiceEffectiveStatementImpl extends
         AbstractEffectiveDocumentedNode<QName, ChoiceStatement> implements
@@ -56,7 +56,7 @@ public class ChoiceEffectiveStatementImpl extends
     ImmutableList<UnknownSchemaNode> unknownNodes;
 
     public ChoiceEffectiveStatementImpl(
-            StmtContext<QName, ChoiceStatement, EffectiveStatement<QName, ChoiceStatement>> ctx) {
+            final StmtContext<QName, ChoiceStatement, EffectiveStatement<QName, ChoiceStatement>> ctx) {
         super(ctx);
 
         this.qname = ctx.getStatementArgument();
@@ -68,7 +68,7 @@ public class ChoiceEffectiveStatementImpl extends
     }
 
     private void initCopyType(
-            StmtContext<QName, ChoiceStatement, EffectiveStatement<QName, ChoiceStatement>> ctx) {
+            final StmtContext<QName, ChoiceStatement, EffectiveStatement<QName, ChoiceStatement>> ctx) {
 
         List<TypeOfCopy> copyTypesFromOriginal = ctx.getCopyHistory();
 
@@ -145,7 +145,7 @@ public class ChoiceEffectiveStatementImpl extends
         this.cases = ImmutableSet.copyOf(casesInit);
     }
 
-    private void resetAugmenting(DataSchemaNode dataSchemaNode) {
+    private static void resetAugmenting(final DataSchemaNode dataSchemaNode) {
         if (dataSchemaNode instanceof LeafEffectiveStatementImpl) {
             LeafEffectiveStatementImpl leaf = (LeafEffectiveStatementImpl) dataSchemaNode;
             leaf.augmenting = false;
index 5b0c94cc98d9c97b31c63a787375d56950150bcc..cebfc0d924f04fd6f74d8ad5e8bf79969cb9e5d8 100644 (file)
@@ -12,14 +12,11 @@ import com.google.common.collect.Collections2;
 import com.google.common.collect.FluentIterable;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Iterables;
-import com.google.common.collect.Lists;
-
 import java.util.Collection;
 import java.util.Collections;
 import java.util.LinkedList;
 import java.util.Map;
 import java.util.NoSuchElementException;
-
 import org.opendaylight.yangtools.yang.model.api.Rfc6020Mapping;
 import org.opendaylight.yangtools.yang.model.api.SchemaNode;
 import org.opendaylight.yangtools.yang.model.api.meta.DeclaredStatement;
@@ -28,7 +25,6 @@ import org.opendaylight.yangtools.yang.model.api.meta.IdentifierNamespace;
 import org.opendaylight.yangtools.yang.model.api.meta.StatementDefinition;
 import org.opendaylight.yangtools.yang.model.api.meta.StatementSource;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.Mutable;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils;
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.StatementContextBase;
 
@@ -43,7 +39,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
 
     private final A argument;
 
-    public EffectiveStatementBase(StmtContext<A, D, ?> ctx) {
+    public EffectiveStatementBase(final StmtContext<A, D, ?> ctx) {
 
         this.stmtCtx = ctx;
         this.statementDefinition = ctx.getPublicDefinition();
@@ -61,7 +57,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
             if(declaredSubstatement.getPublicDefinition() == Rfc6020Mapping.USES) {
                 substatementsInit.add(declaredSubstatement);
                 substatementsInit.addAll(declaredSubstatement.getEffectOfStatement());
-                ((StatementContextBase)ctx).removeStatementsFromEffectiveSubstatements(declaredSubstatement
+                ((StatementContextBase<?, ?, ?>)ctx).removeStatementsFromEffectiveSubstatements(declaredSubstatement
                         .getEffectOfStatement());
             } else {
                 substatementsInit.add(declaredSubstatement);
@@ -99,14 +95,14 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
 
     @Override
     public <K, V, N extends IdentifierNamespace<K, V>> V get(
-            Class<N> namespace, K identifier) {
+            final Class<N> namespace, final K identifier) {
         return stmtCtx.getFromNamespace(namespace, identifier);
     }
 
     @Override
     public <K, V, N extends IdentifierNamespace<K, V>> Map<K, V> getAll(
-            Class<N> namespace) {
-        return (Map<K, V>) stmtCtx.getAllFromNamespace(namespace);
+            final Class<N> namespace) {
+        return stmtCtx.getAllFromNamespace(namespace);
     }
 
     @Override
@@ -119,7 +115,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
     }
 
     protected final <S extends EffectiveStatement<?, ?>> S firstEffective(
-            Class<S> type) {
+            final Class<S> type) {
         S result = null;
         try {
             result = type.cast(Iterables.find(substatements,
@@ -130,7 +126,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
         return result;
     }
 
-    protected final <S extends SchemaNode> S firstSchemaNode(Class<S> type) {
+    protected final <S extends SchemaNode> S firstSchemaNode(final Class<S> type) {
         S result = null;
         try {
             result = type.cast(Iterables.find(substatements,
@@ -143,7 +139,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
 
     @SuppressWarnings("unchecked")
     protected final <T> Collection<T> allSubstatementsOfType(
-            Class<T> type) {
+            final Class<T> type) {
         Collection<T> result = null;
 
         try {
@@ -155,7 +151,7 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
         return result;
     }
 
-    protected final <T> T firstSubstatementOfType(Class<T> type) {
+    protected final <T> T firstSubstatementOfType(final Class<T> type) {
         T result = null;
         try {
             result = type.cast(Iterables.find(substatements,
@@ -166,8 +162,8 @@ abstract public class EffectiveStatementBase<A, D extends DeclaredStatement<A>>
         return result;
     }
 
-    protected final <R> R firstSubstatementOfType(Class<?> type,
-            Class<R> returnType) {
+    protected final <R> R firstSubstatementOfType(final Class<?> type,
+            final Class<R> returnType) {
         R result = null;
         try {
             result = returnType.cast(Iterables.find(
index 8ae39ae7de07e4fc1ea652865382a2becffb7257..9c067fbea0a7eefaea08d55b71f2eefdb8203ff9 100644 (file)
@@ -67,7 +67,7 @@ public class ExtendedTypeEffectiveStatementImpl extends EffectiveStatementBase<S
     private final boolean isExtended;
 
     public ExtendedTypeEffectiveStatementImpl(
-            StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx, boolean isExtended) {
+            final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx, final boolean isExtended) {
         super(ctx);
 
         this.isExtended = isExtended;
@@ -90,7 +90,7 @@ public class ExtendedTypeEffectiveStatementImpl extends EffectiveStatementBase<S
         validateTypeConstraints(ctx);
     }
 
-    private QName initQName(final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx,
+    private static QName initQName(final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx,
             final boolean isExtended) {
 
         QName qName;
@@ -117,7 +117,7 @@ public class ExtendedTypeEffectiveStatementImpl extends EffectiveStatementBase<S
         return qName;
     }
 
-    private TypeDefinition<?> parseBaseTypeFromCtx(
+    private static TypeDefinition<?> parseBaseTypeFromCtx(
             final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx) {
 
         TypeDefinition<?> baseType;
@@ -155,7 +155,7 @@ public class ExtendedTypeEffectiveStatementImpl extends EffectiveStatementBase<S
         typeConstraints.validateConstraints();
     }
 
-    private TypeConstraints addConstraintsFromBaseType(final TypeConstraints typeConstraints,
+    private static TypeConstraints addConstraintsFromBaseType(final TypeConstraints typeConstraints,
             final TypeDefinition<?> baseType) {
 
         final String baseTypeName = baseType.getQName().getLocalName();
index ebbac6d177c55c3b7c333f2f11fef58055c19fe8..a6aeb7ca92147ecb65fe176c5788b4401f567cfd 100644 (file)
@@ -7,51 +7,50 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective;
 
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
-
-import org.opendaylight.yangtools.yang.parser.spi.source.ModuleCtxToModuleQName;
-import org.opendaylight.yangtools.yang.model.util.ModuleImportImpl;
-import org.opendaylight.yangtools.yang.model.api.stmt.ModuleStatement;
 import com.google.common.collect.ImmutableList;
-import com.google.common.collect.ImmutableSet;
-import org.opendaylight.yangtools.yang.model.util.ExtendedType;
-import java.util.LinkedHashSet;
-import java.util.LinkedHashMap;
 import com.google.common.collect.ImmutableMap;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.GroupingDefinition;
-import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import org.opendaylight.yangtools.yang.model.api.UsesNode;
-import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
-import org.opendaylight.yangtools.yang.model.api.stmt.SubmoduleStatement;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.Mutable;
-import org.opendaylight.yangtools.yang.parser.spi.SubmoduleNamespace;
-import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
-import java.util.Map;
-import org.opendaylight.yangtools.yang.parser.spi.source.DeclarationInTextSource;
-import org.opendaylight.yangtools.yang.parser.spi.source.IncludedSubmoduleNameToIdentifier;
+import com.google.common.collect.ImmutableSet;
 import java.net.URI;
 import java.util.Collection;
 import java.util.Date;
 import java.util.HashSet;
+import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
 import java.util.LinkedList;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
 import org.opendaylight.yangtools.concepts.Immutable;
+import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchema;
+import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Deviation;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
 import org.opendaylight.yangtools.yang.model.api.FeatureDefinition;
+import org.opendaylight.yangtools.yang.model.api.GroupingDefinition;
 import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
+import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
 import org.opendaylight.yangtools.yang.model.api.ModuleImport;
 import org.opendaylight.yangtools.yang.model.api.NotificationDefinition;
 import org.opendaylight.yangtools.yang.model.api.RpcDefinition;
+import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.UsesNode;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.ModuleStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.SubmoduleStatement;
+import org.opendaylight.yangtools.yang.model.util.ExtendedType;
+import org.opendaylight.yangtools.yang.model.util.ModuleImportImpl;
+import org.opendaylight.yangtools.yang.parser.spi.SubmoduleNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
+import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.Mutable;
+import org.opendaylight.yangtools.yang.parser.spi.source.DeclarationInTextSource;
+import org.opendaylight.yangtools.yang.parser.spi.source.IncludedSubmoduleNameToIdentifier;
+import org.opendaylight.yangtools.yang.parser.spi.source.ModuleCtxToModuleQName;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
 
 public class ModuleEffectiveStatementImpl extends
         AbstractEffectiveDocumentedNode<String, ModuleStatement> implements
@@ -59,7 +58,7 @@ public class ModuleEffectiveStatementImpl extends
 
     private final QNameModule qNameModule;
     private final String name;
-    private String sourcePath;
+    private final String sourcePath;
     private String prefix;
     private String yangVersion;
     private String organization;
@@ -74,7 +73,7 @@ public class ModuleEffectiveStatementImpl extends
     private ImmutableList<ExtensionDefinition> extensionNodes;
     private ImmutableSet<IdentitySchemaNode> identities;
     private ImmutableList<UnknownSchemaNode> unknownNodes;
-    private String source;
+    private final String source;
     private ImmutableList<EffectiveStatement<?, ?>> substatementsOfSubmodules;
 
     private ImmutableMap<QName, DataSchemaNode> childNodes;
@@ -84,7 +83,7 @@ public class ModuleEffectiveStatementImpl extends
     private ImmutableSet<DataSchemaNode> publicChildNodes;
 
     public ModuleEffectiveStatementImpl(
-            StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
+            final StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
         super(ctx);
 
         name = argument();
@@ -127,7 +126,7 @@ public class ModuleEffectiveStatementImpl extends
     }
 
     private void initSubmodules(
-            StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
+            final StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
         Map<String, ModuleIdentifier> includedSubmodulesMap = ctx
                 .getAllFromCurrentStmtCtxNamespace(IncludedSubmoduleNameToIdentifier.class);
 
@@ -159,7 +158,7 @@ public class ModuleEffectiveStatementImpl extends
     }
 
     private void initSubstatementCollections(
-            StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
+            final StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
         List<EffectiveStatement<?, ?>> effectiveSubstatements = new LinkedList<>();
 
         effectiveSubstatements.addAll(effectiveSubstatements());
@@ -274,9 +273,9 @@ public class ModuleEffectiveStatementImpl extends
         this.uses = ImmutableSet.copyOf(mutableUses);
     }
 
-    private Set<ModuleImport> resolveModuleImports(
-            Set<ModuleImport> importsInit,
-            StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
+    private static Set<ModuleImport> resolveModuleImports(
+            final Set<ModuleImport> importsInit,
+            final StmtContext<String, ModuleStatement, EffectiveStatement<String, ModuleStatement>> ctx) {
         Set<ModuleImport> resolvedModuleImports = new LinkedHashSet<>();
         for (ModuleImport moduleImport : importsInit) {
             if (moduleImport.getRevision().equals(
index b530c55c50c60dbd153cc36364848979644a3d0d..0ad5b51fc30eb97e91641bdf048c60205fa9568b 100644 (file)
@@ -7,52 +7,51 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective;
 
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
-
-import org.opendaylight.yangtools.yang.model.util.ModuleImportImpl;
-import org.opendaylight.yangtools.yang.model.util.ExtendedType;
-import java.util.LinkedHashSet;
-import java.util.LinkedHashMap;
+import static org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils.firstAttributeOf;
+import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.GroupingDefinition;
-import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import org.opendaylight.yangtools.yang.model.api.UsesNode;
-import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
-import java.util.Map;
-import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
-import org.opendaylight.yangtools.yang.parser.spi.SubmoduleNamespace;
-import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.Mutable;
-import org.opendaylight.yangtools.yang.parser.spi.source.DeclarationInTextSource;
-import org.opendaylight.yangtools.yang.parser.spi.source.IncludedSubmoduleNameToIdentifier;
+import com.google.common.collect.ImmutableSet;
 import java.net.URI;
 import java.util.Collection;
 import java.util.Date;
 import java.util.HashSet;
+import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
 import java.util.LinkedList;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
 import org.opendaylight.yangtools.concepts.Immutable;
+import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.AugmentationSchema;
+import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Deviation;
 import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
 import org.opendaylight.yangtools.yang.model.api.FeatureDefinition;
+import org.opendaylight.yangtools.yang.model.api.GroupingDefinition;
 import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
+import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
 import org.opendaylight.yangtools.yang.model.api.ModuleImport;
 import org.opendaylight.yangtools.yang.model.api.NotificationDefinition;
 import org.opendaylight.yangtools.yang.model.api.RpcDefinition;
+import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.UsesNode;
 import org.opendaylight.yangtools.yang.model.api.meta.EffectiveStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.BelongsToStatement;
 import org.opendaylight.yangtools.yang.model.api.stmt.SubmoduleStatement;
+import org.opendaylight.yangtools.yang.model.util.ExtendedType;
+import org.opendaylight.yangtools.yang.model.util.ModuleImportImpl;
+import org.opendaylight.yangtools.yang.parser.spi.SubmoduleNamespace;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
-import com.google.common.collect.ImmutableList;
-import com.google.common.collect.ImmutableSet;
+import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext.Mutable;
+import org.opendaylight.yangtools.yang.parser.spi.source.DeclarationInTextSource;
+import org.opendaylight.yangtools.yang.parser.spi.source.IncludedSubmoduleNameToIdentifier;
 import org.opendaylight.yangtools.yang.parser.spi.source.ModuleNameToModuleQName;
-import static org.opendaylight.yangtools.yang.parser.spi.meta.StmtContextUtils.firstAttributeOf;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
 
 public class SubmoduleEffectiveStatementImpl
         extends
@@ -61,7 +60,7 @@ public class SubmoduleEffectiveStatementImpl
 
     private final QNameModule qNameModule;
     private final String name;
-    private String sourcePath;
+    private final String sourcePath;
     private String prefix;
     private String yangVersion;
     private String organization;
@@ -86,7 +85,7 @@ public class SubmoduleEffectiveStatementImpl
     private ImmutableSet<DataSchemaNode> publicChildNodes;
 
     public SubmoduleEffectiveStatementImpl(
-            StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
+            final StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
         super(ctx);
 
         name = argument();
@@ -131,7 +130,7 @@ public class SubmoduleEffectiveStatementImpl
     }
 
     private void initSubmodules(
-            StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
+            final StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
         Map<String, ModuleIdentifier> includedSubmodulesMap = ctx
                 .getAllFromCurrentStmtCtxNamespace(IncludedSubmoduleNameToIdentifier.class);
 
@@ -159,7 +158,7 @@ public class SubmoduleEffectiveStatementImpl
         this.substatementsOfSubmodules = ImmutableList.copyOf(substatementsOfSubmodulesInit);
     }
 
-    private void initSubstatementCollections(StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
+    private void initSubstatementCollections(final StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
         List<EffectiveStatement<?, ?>> effectiveSubstatements = new LinkedList<>();
 
         effectiveSubstatements.addAll(effectiveSubstatements());
@@ -266,9 +265,9 @@ public class SubmoduleEffectiveStatementImpl
         this.uses = ImmutableSet.copyOf(mutableUses);
     }
 
-    private Set<ModuleImport> resolveModuleImports(
-            Set<ModuleImport> importsInit,
-            StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
+    private static Set<ModuleImport> resolveModuleImports(
+            final Set<ModuleImport> importsInit,
+            final StmtContext<String, SubmoduleStatement, EffectiveStatement<String, SubmoduleStatement>> ctx) {
         Set<ModuleImport> resolvedModuleImports = new LinkedHashSet<>();
         for (ModuleImport moduleImport : importsInit) {
             if (moduleImport.getRevision().equals(
index 821edb9fa3ca8e2b39ee3e5b01a7406234c15a11..e20dd839c54a439d05031a6191fa74b4da26159a 100644 (file)
@@ -8,8 +8,6 @@
 
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type;
 
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveStatementBase;
-
 import com.google.common.base.Optional;
 import java.util.Collections;
 import java.util.List;
@@ -23,6 +21,7 @@ import org.opendaylight.yangtools.yang.model.api.stmt.TypeStatement;
 import org.opendaylight.yangtools.yang.model.api.type.BinaryTypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.LengthConstraint;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveStatementBase;
 
 public class BinaryEffectiveStatementImpl extends
         EffectiveStatementBase<String, TypeStatement> implements
@@ -32,10 +31,9 @@ public class BinaryEffectiveStatementImpl extends
     private static final String REFERENCE = "https://tools.ietf.org/html/rfc6020#section-9.8";
     private static final String UNITS = "";
 
-    private final static QName QNAME = QName.create(
-            YangConstants.RFC6020_YANG_MODULE, "binary");
-
-    private final static SchemaPath PATH = SchemaPath.create(true, QNAME);
+    private static final QName QNAME = QName.create(YangConstants.RFC6020_YANG_MODULE, "binary");
+    private static final SchemaPath PATH = SchemaPath.create(true, QNAME);
+    private static final Optional<String> OPTIONAL_EMPTY = Optional.of("");
     private final List<Byte> defaultValue = Collections.emptyList();
     private final List<LengthConstraint> lengthConstraints;
 
@@ -44,7 +42,7 @@ public class BinaryEffectiveStatementImpl extends
         super(ctx);
 
         final LengthConstraint lengthConstraint = new LengthConstraintEffectiveImpl(
-                0, Long.MAX_VALUE, Optional.of(""), Optional.of(""));
+                0, Long.MAX_VALUE, OPTIONAL_EMPTY, OPTIONAL_EMPTY);
         lengthConstraints = Collections.singletonList(lengthConstraint);
     }
 
index 6dddaafa30b9fe70a897e255123ca0670ac95bf0..a762abd8f494934c3f5680b807c26ce06e312161 100644 (file)
@@ -7,29 +7,28 @@
  */
 package org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.type;
 
-import org.opendaylight.yangtools.yang.model.util.ExtendedType;
-import org.opendaylight.yangtools.yang.model.util.ExtendedType.Builder;
-import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveStatementBase;
-import org.opendaylight.yangtools.yang.model.util.Decimal64;
-import org.opendaylight.yangtools.yang.model.api.stmt.TypeStatement;
 import com.google.common.base.Optional;
 import com.google.common.collect.ImmutableList;
 import java.math.BigDecimal;
-import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.YangConstants;
 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.meta.EffectiveStatement;
+import org.opendaylight.yangtools.yang.model.api.stmt.TypeStatement;
 import org.opendaylight.yangtools.yang.model.api.type.DecimalTypeDefinition;
 import org.opendaylight.yangtools.yang.model.api.type.RangeConstraint;
+import org.opendaylight.yangtools.yang.model.util.Decimal64;
+import org.opendaylight.yangtools.yang.model.util.ExtendedType;
+import org.opendaylight.yangtools.yang.model.util.ExtendedType.Builder;
 import org.opendaylight.yangtools.yang.parser.spi.meta.StmtContext;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.TypeUtils;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.Utils;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveStatementBase;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.FractionDigitsEffectiveStatementImpl;
 
 public class Decimal64SpecificationEffectiveStatementImpl extends
@@ -51,6 +50,16 @@ public class Decimal64SpecificationEffectiveStatementImpl extends
             "-922337203685477580.8");
     private static final BigDecimal MAX_VALUE = new BigDecimal(
             "922337203685477580.7");
+    private static final List<RangeConstraint> DEFAULT_RANGE_STATEMENTS;
+    static {
+        final String rangeDescription = "Integer values between " + MIN_VALUE
+                + " and " + MAX_VALUE + ", inclusively.";
+        final String rangeReference = RangeConstraintEffectiveImpl.DEFAULT_REFERENCE;
+
+        DEFAULT_RANGE_STATEMENTS = ImmutableList.<RangeConstraint>of(
+                new RangeConstraintEffectiveImpl(MIN_VALUE, MAX_VALUE, Optional.of(rangeDescription),
+                Optional.of(rangeReference)));
+    }
 
     private List<RangeConstraint> rangeConstraints;
     private Integer fractionDigits;
@@ -61,7 +70,7 @@ public class Decimal64SpecificationEffectiveStatementImpl extends
     private final boolean isExtended;
 
     public Decimal64SpecificationEffectiveStatementImpl(
-            StmtContext<String, TypeStatement.Decimal64Specification, EffectiveStatement<String, TypeStatement.Decimal64Specification>> ctx) {
+            final StmtContext<String, TypeStatement.Decimal64Specification, EffectiveStatement<String, TypeStatement.Decimal64Specification>> ctx) {
         super(ctx);
 
         for (final EffectiveStatement<?, ?> effectiveStatement : effectiveSubstatements()) {
@@ -81,12 +90,12 @@ public class Decimal64SpecificationEffectiveStatementImpl extends
             path = parentPath.createChild(extendedTypeQName);
         } else {
             isExtended = false;
-            rangeConstraints = defaultRangeStatements();
+            rangeConstraints = DEFAULT_RANGE_STATEMENTS;
             path = Utils.getSchemaPath(ctx.getParentContext()).createChild(QNAME);
         }
     }
 
-    private boolean validateRanges(List<RangeConstraint> initRanges) {
+    private static boolean validateRanges(final List<RangeConstraint> initRanges) {
         for (RangeConstraint rangeConstraint : initRanges) {
 
             String maxValueString = rangeConstraint.getMax().toString();
@@ -108,19 +117,6 @@ public class Decimal64SpecificationEffectiveStatementImpl extends
                 : Collections.<RangeConstraint> emptyList();
     }
 
-    private List<RangeConstraint> defaultRangeStatements() {
-
-        final List<RangeConstraint> rangeStmts = new ArrayList<>();
-        final String rangeDescription = "Integer values between " + MIN_VALUE
-                + " and " + MAX_VALUE + ", inclusively.";
-        final String rangeReference = RangeConstraintEffectiveImpl.DEFAULT_REFERENCE;
-
-        rangeStmts.add(new RangeConstraintEffectiveImpl(MIN_VALUE, MAX_VALUE,
-                Optional.of(rangeDescription), Optional.of(rangeReference)));
-
-        return ImmutableList.copyOf(rangeStmts);
-    }
-
     public boolean isExtended() {
         return isExtended;
     }
index 612b7253c3b5843f70ee8e0ec2d3eccbc293c529..8a0b14c864ed92447ea30c175a622347c359221a 100644 (file)
@@ -36,16 +36,18 @@ public class StringEffectiveStatementImpl extends EffectiveStatementBase<String,
             + "Legal characters are tab, carriage return, line feed, and the legal "
             + "characters of Unicode and ISO/IEC 10646";
     private static final String REFERENCE = "https://tools.ietf.org/html/rfc6020#section-9.4";
+    private static final String UNITS = "";
+    private static final Optional<String> OPTIONAL_EMPTY = Optional.of("");
+
     private final List<LengthConstraint> lengthConstraints;
     private final List<PatternConstraint> patternConstraints;
-    private static final String UNITS = "";
 
     public StringEffectiveStatementImpl(
-            StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx) {
+            final StmtContext<String, TypeStatement, EffectiveStatement<String, TypeStatement>> ctx) {
         super(ctx);
 
-        final LengthConstraint defLength = new LengthConstraintEffectiveImpl(0, Integer.MAX_VALUE, Optional.of(""),
-                Optional.of(""));
+        final LengthConstraint defLength = new LengthConstraintEffectiveImpl(0, Integer.MAX_VALUE, OPTIONAL_EMPTY,
+            OPTIONAL_EMPTY);
 
         lengthConstraints = Collections.singletonList(defLength);
         patternConstraints = Collections.emptyList();
index 50c94f99999d5eccb49563aa4d6ee66c39d0b8f9..1e19ea258760cd80c52b2265be2a2ad1ac3dc747 100644 (file)
@@ -7,38 +7,10 @@
  */
 package org.opendaylight.yangtools.yang.parser.impl;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
-
 import java.io.File;
-import java.math.BigInteger;
-import java.net.URI;
 import java.util.Arrays;
-import java.util.List;
-import java.util.Set;
-import org.junit.Before;
 import org.junit.Test;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.model.api.IdentitySchemaNode;
-import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.Module;
-import org.opendaylight.yangtools.yang.model.api.Status;
-import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import org.opendaylight.yangtools.yang.model.api.type.BitsTypeDefinition.Bit;
-import org.opendaylight.yangtools.yang.model.api.type.EnumTypeDefinition.EnumPair;
-import org.opendaylight.yangtools.yang.model.api.type.LengthConstraint;
-import org.opendaylight.yangtools.yang.model.api.type.PatternConstraint;
-import org.opendaylight.yangtools.yang.model.api.type.StringTypeDefinition;
 import org.opendaylight.yangtools.yang.model.parser.api.YangContextParser;
-import org.opendaylight.yangtools.yang.model.util.BitsType;
-import org.opendaylight.yangtools.yang.model.util.EnumerationType;
-import org.opendaylight.yangtools.yang.model.util.ExtendedType;
-import org.opendaylight.yangtools.yang.model.util.IdentityrefType;
-import org.opendaylight.yangtools.yang.model.util.InstanceIdentifierType;
-import org.opendaylight.yangtools.yang.model.util.UnionType;
 
 public class Bug2291Test {
 
index c1f164e053d8f39db4f0c7929b04e61c97218e14..31e28fdb2d867354d0d7b324d7a2b54372feaed5 100644 (file)
@@ -3,22 +3,21 @@ package org.opendaylight.yangtools.yang.stmt.effective.build.test;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
-
-import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
-import java.util.Collection;
-import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import java.util.Set;
-import java.net.URISyntaxException;
 import java.io.FileNotFoundException;
-import org.opendaylight.yangtools.yang.model.api.SchemaContext;
-import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import java.net.URI;
+import java.net.URISyntaxException;
+import java.util.Collection;
+import java.util.Set;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.GroupingDefinition;
+import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
+import org.opendaylight.yangtools.yang.model.api.SchemaContext;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
@@ -129,8 +128,7 @@ public class EffectiveBuildTest {
         assertNotNull(result);
     }
 
-    private void addSources(BuildAction reactor,
-            YangStatementSourceImpl... sources) {
+    private static void addSources(final BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index 9de3b4b04f735fb3145977f3dcac89b21ce1cee3..3833dcf0758e868c98ce2d22f1b150aa8b0ef452 100644 (file)
@@ -3,25 +3,23 @@ package org.opendaylight.yangtools.yang.stmt.effective.build.test;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
-
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.util.Collection;
+import java.util.Set;
+import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
-
 import org.opendaylight.yangtools.yang.common.QNameModule;
-import java.net.URISyntaxException;
-import java.net.URI;
 import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
-import java.util.Collection;
 import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import java.util.Set;
 import org.opendaylight.yangtools.yang.model.api.Module;
-import org.opendaylight.yangtools.yang.stmt.test.StmtTestUtils;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementReactor;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangInferencePipeline;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangStatementSourceImpl;
-import org.junit.Test;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
+import org.opendaylight.yangtools.yang.stmt.test.StmtTestUtils;
 
 public class EffectiveModulesAndSubmodulesTest {
 
@@ -160,16 +158,16 @@ public class EffectiveModulesAndSubmodulesTest {
 
     }
 
-    private void getDataChildByNameSubTest(EffectiveSchemaContext result,
-            Module root) {
+    private static void getDataChildByNameSubTest(final EffectiveSchemaContext result,
+            final Module root) {
         DataSchemaNode containerInRoot = result.getDataChildByName(QName
                 .create(root.getQNameModule(), "container-in-root-module"));
         assertNotNull(containerInRoot);
         assertEquals("desc", containerInRoot.getDescription());
     }
 
-    private void findModulesSubTest(EffectiveSchemaContext result, Module root,
-            Module imported) throws URISyntaxException {
+    private static void findModulesSubTest(final EffectiveSchemaContext result, final Module root,
+            final Module imported) throws URISyntaxException {
         Module foundRoot = result.findModuleByName("root-module",
                 SimpleDateFormatUtil.DEFAULT_DATE_REV);
         Set<Module> foundRoots = result.findModuleByNamespace(new URI(
index f24652f444815fd094642c3f780475f4d222c263..9d3b8f490e4cd9612fe995f4813d4104019eebc0 100644 (file)
@@ -4,20 +4,19 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
-
-import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
-
+import java.net.URISyntaxException;
+import java.util.Set;
+import org.junit.Test;
+import org.opendaylight.yangtools.yang.common.QName;
+import org.opendaylight.yangtools.yang.common.QNameModule;
 import org.opendaylight.yangtools.yang.model.api.ChoiceSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ConstraintDefinition;
+import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.SchemaNode;
-import org.opendaylight.yangtools.yang.model.util.SchemaContextUtil;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.common.QNameModule;
-import java.util.Set;
-import org.opendaylight.yangtools.yang.model.api.Module;
-import java.net.URISyntaxException;
+import org.opendaylight.yangtools.yang.model.util.SchemaContextUtil;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementReactor;
@@ -25,7 +24,6 @@ import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangInferencePipeline
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangStatementSourceImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 import org.opendaylight.yangtools.yang.stmt.test.StmtTestUtils;
-import org.junit.Test;
 
 public class EffectiveUsesRefineAndConstraintsTest {
 
@@ -88,8 +86,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
 
     }
 
-    private void checkOriginalContainer(EffectiveSchemaContext result,
-            SchemaPath path) {
+    private static void checkOriginalContainer(final EffectiveSchemaContext result,
+            final SchemaPath path) {
         SchemaNode containerInContainerNode = SchemaContextUtil
                 .findDataSchemaNode(result, path);
         assertNotNull(containerInContainerNode);
@@ -105,8 +103,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertEquals(0, containerConstraints.getMustConstraints().size());
     }
 
-    private void checkOriginalChoice(EffectiveSchemaContext result,
-            SchemaPath path) {
+    private static void checkOriginalChoice(final EffectiveSchemaContext result,
+            final SchemaPath path) {
         SchemaNode choiceInContainerNode = SchemaContextUtil
                 .findDataSchemaNode(result, path);
         assertNotNull(choiceInContainerNode);
@@ -119,8 +117,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertTrue(choiceConstraints.getMustConstraints().isEmpty());
     }
 
-    private void checkOriginalList(EffectiveSchemaContext result,
-            SchemaPath path) {
+    private static void checkOriginalList(final EffectiveSchemaContext result,
+            final SchemaPath path) {
         SchemaNode listInContainerNode = SchemaContextUtil.findDataSchemaNode(
                 result, path);
         assertNotNull(listInContainerNode);
@@ -136,8 +134,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertEquals(1, listConstraints.getMustConstraints().size());
     }
 
-    private void checkRefinedContainer(EffectiveSchemaContext result,
-            SchemaPath path) {
+    private static void checkRefinedContainer(final EffectiveSchemaContext result,
+            final SchemaPath path) {
         SchemaNode containerInContainerNode = SchemaContextUtil
                 .findDataSchemaNode(result, path);
         assertNotNull(containerInContainerNode);
@@ -153,8 +151,8 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertEquals(1, containerConstraints.getMustConstraints().size());
     }
 
-    private void checkRefinedChoice(EffectiveSchemaContext result,
-            SchemaPath path) {
+    private static void checkRefinedChoice(final EffectiveSchemaContext result,
+            final SchemaPath path) {
         SchemaNode choiceInContainerNode = SchemaContextUtil
                 .findDataSchemaNode(result, path);
         assertNotNull(choiceInContainerNode);
@@ -167,7 +165,7 @@ public class EffectiveUsesRefineAndConstraintsTest {
         assertTrue(choiceConstraints.getMustConstraints().isEmpty());
     }
 
-    private void checkRefinedList(EffectiveSchemaContext result, SchemaPath path) {
+    private static void checkRefinedList(final EffectiveSchemaContext result, final SchemaPath path) {
         SchemaNode listInContainerNode = SchemaContextUtil.findDataSchemaNode(
                 result, path);
         assertNotNull(listInContainerNode);
index 3073f3dfacffcd579140c7f96c9301938ed2308f..7cdbb128a74ca6f52a7ffb0661d69dc6cb57bd41 100644 (file)
@@ -108,7 +108,7 @@ public class ListKeysTest {
         }
     }
 
-    private void addSources(CrossSourceStatementReactor.BuildAction reactor, YangStatementSourceImpl... sources) {
+    private static void addSources(final CrossSourceStatementReactor.BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index c42bbb95f5098247cf755b2ef94636adbe47ad80..d6677a9339c715b75812027adbe83ebfdc8fef3a 100644 (file)
@@ -7,6 +7,13 @@
  */
 package org.opendaylight.yangtools.yang.stmt.retest;
 
+import static org.mockito.Mockito.doReturn;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.text.ParseException;
+import java.util.Collections;
+import java.util.Date;
+import java.util.Map;
 import org.junit.Before;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
@@ -15,16 +22,6 @@ import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.ModuleIdentifier;
 //import org.opendaylight.yangtools.yang.parser.impl.SchemaContextImpl;
 
-import java.net.URI;
-import java.net.URISyntaxException;
-import java.text.ParseException;
-import java.util.Collections;
-import java.util.Date;
-import java.util.Map;
-
-import static org.junit.Assert.assertEquals;
-import static org.mockito.Mockito.doReturn;
-
 public class SchemaContextTest {
     @Mock
     private Module oldModule;
index 44a3bbcf4cbb3d4b754b9f7edb3c1803c539b5c7..94b4464139f6e8851bab6e1c21044584eea23342 100644 (file)
@@ -13,7 +13,6 @@ import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
-
 import java.io.File;
 import java.math.BigInteger;
 import java.net.URI;
@@ -383,7 +382,7 @@ public class TypesResolutionTest {
         }
     }
 
-    private void addSources(CrossSourceStatementReactor.BuildAction reactor, YangStatementSourceImpl... sources) {
+    private static void addSources(final CrossSourceStatementReactor.BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index fc29bc772802947b3dd98ee8de0b7e6854c8f337..98f819fc95d85161e77f58815528dad5735e27be 100644 (file)
@@ -10,7 +10,6 @@ package org.opendaylight.yangtools.yang.stmt.retest;
 import static org.hamcrest.CoreMatchers.containsString;
 import static org.junit.Assert.assertThat;
 import static org.mockito.Mockito.mock;
-
 import org.antlr.v4.runtime.tree.ParseTree;
 import org.junit.Before;
 import org.junit.Test;
@@ -77,7 +76,7 @@ public class YangModelValidationSubModuleTest {
         }
     }
 
-    private Submodule_stmtContext mockSubmoduleParent(final ParseTree child,
+    private static Submodule_stmtContext mockSubmoduleParent(final ParseTree child,
             final String moduleName) {
         Submodule_stmtContext ctx = YangModelValidationTest.mockStatement(
                 Submodule_stmtContext.class, moduleName);
index 7f3fa64c00a7c516b359a7cc55990a970b07169c..2cc4f1530499b79508e04f78f3107631ae4a643d 100644 (file)
@@ -8,17 +8,14 @@
 package org.opendaylight.yangtools.yang.stmt.retest;
 
 import static org.hamcrest.CoreMatchers.containsString;
-import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
 import static org.junit.Assert.fail;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
-
 import java.text.SimpleDateFormat;
 import java.util.Collections;
 import java.util.Date;
-
 import org.antlr.v4.runtime.Token;
 import org.antlr.v4.runtime.tree.ParseTree;
 import org.antlr.v4.runtime.tree.TerminalNode;
@@ -211,7 +208,7 @@ public class YangModelValidationTest {
         }
     }
 
-    private Import_stmtContext mockImport(final String name, final String prefixName) {
+    private static Import_stmtContext mockImport(final String name, final String prefixName) {
         Import_stmtContext impor = mockStatement(Import_stmtContext.class, name);
 
         Prefix_stmtContext prefix = mockStatement(Prefix_stmtContext.class, prefixName);
@@ -226,7 +223,7 @@ public class YangModelValidationTest {
         return new SimpleDateFormat("yyyy-MM-dd").format(new Date());
     }
 
-    private Include_stmtContext mockInclude(final String name) {
+    private static Include_stmtContext mockInclude(final String name) {
         Include_stmtContext incl = mockStatement(Include_stmtContext.class, name);
 
         Revision_date_stmtContext revDate = mockStatement(Revision_date_stmtContext.class, getFormattedDate());
index 7d1e329bebb79829d2c3e3e480c837f284a36046..7e652bf1a984b153351a32e52b168ec6c28ba210 100644 (file)
@@ -887,7 +887,7 @@ public class YangParserTest {
     // assertSetEquals(newModules, ctx.getModules());
     // }
 
-    private void checkOrder(final Collection<Module> modules) {
+    private static void checkOrder(final Collection<Module> modules) {
         Iterator<Module> it = modules.iterator();
         Module m = it.next();
         assertEquals("m2", m.getName());
@@ -907,7 +907,7 @@ public class YangParserTest {
         assertEquals("m1", m.getName());
     }
 
-    private void assertSetEquals(final Set<Module> s1, final Set<Module> s2) {
+    private static void assertSetEquals(final Set<Module> s1, final Set<Module> s2) {
         assertEquals(s1, s2);
         Iterator<Module> it = s1.iterator();
         for (Module m : s2) {
@@ -980,7 +980,7 @@ public class YangParserTest {
         }
     }
 
-    private void addSources(CrossSourceStatementReactor.BuildAction reactor, YangStatementSourceImpl... sources) {
+    private static void addSources(final CrossSourceStatementReactor.BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index e9e7eb176308830564f89c3dd41dd8d9060e012d..1ca6bef34c0b5e150d0e32084e474e04b2625443 100644 (file)
@@ -7,32 +7,32 @@
  */
 package org.opendaylight.yangtools.yang.stmt.test;
 
-import static org.junit.Assert.*;
-
-import org.opendaylight.yangtools.yang.model.api.DataNodeContainer;
-
-import org.opendaylight.yangtools.yang.model.api.UsesNode;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+import java.io.FileNotFoundException;
 import java.net.URI;
-import org.opendaylight.yangtools.yang.common.QNameModule;
-import org.opendaylight.yangtools.yang.common.QName;
-import org.opendaylight.yangtools.yang.model.api.ChoiceSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
+import java.net.URISyntaxException;
+import java.text.ParseException;
 import java.util.List;
-import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
+import java.util.Set;
+import org.junit.Test;
+import org.opendaylight.yangtools.yang.common.QName;
+import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
+import org.opendaylight.yangtools.yang.model.api.AugmentationSchema;
 import org.opendaylight.yangtools.yang.model.api.ChoiceCaseNode;
+import org.opendaylight.yangtools.yang.model.api.ChoiceSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.ContainerSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.DataNodeContainer;
 import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import java.util.Set;
-import org.opendaylight.yangtools.yang.model.api.AugmentationSchema;
-import java.text.ParseException;
+import org.opendaylight.yangtools.yang.model.api.ListSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
-import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
-import java.io.FileNotFoundException;
-import java.net.URISyntaxException;
+import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.UsesNode;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
-import org.junit.Test;
 
 public class ControllerStmtParserTest {
 
@@ -47,7 +47,7 @@ public class ControllerStmtParserTest {
         configModuleTest(context);
     }
 
-    private void salDomBrokerImplModuleTest(SchemaContext context)
+    private static void salDomBrokerImplModuleTest(final SchemaContext context)
             throws ParseException {
         Module module = context.findModuleByName(
                 "opendaylight-sal-dom-broker-impl", SimpleDateFormatUtil
@@ -85,7 +85,7 @@ public class ControllerStmtParserTest {
         assertTrue(checked);
     }
 
-    private void configModuleTest(SchemaContext context) throws ParseException,
+    private static void configModuleTest(final SchemaContext context) throws ParseException,
             URISyntaxException {
         Module configModule = context.findModuleByName("config",
                 SimpleDateFormatUtil.getRevisionFormat().parse("2013-04-05"));
@@ -166,7 +166,7 @@ public class ControllerStmtParserTest {
                 .getPath().getLastComponent());
     }
 
-    private int getChildNodeSizeWithoutUses(final DataNodeContainer csn) {
+    private static int getChildNodeSizeWithoutUses(final DataNodeContainer csn) {
         int result = 0;
         for (DataSchemaNode dsn : csn.getChildNodes()) {
             if (dsn.isAddedByUses() == false) {
index 8501ae421786a924cf7d6db4d6a0e89bb4876ca9..cd56e7e472a873549f6ac97a69cad1cb0674d100 100644 (file)
@@ -12,9 +12,7 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
-
 import org.junit.Test;
-import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ModelProcessingPhase;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.meta.SomeModifiersUnresolvedException;
@@ -23,6 +21,7 @@ import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementR
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.EffectiveModelContext;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangInferencePipeline;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangStatementSourceImpl;
+import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 
 public class ImportResolutionBasicTest {
 
@@ -109,7 +108,7 @@ public class ImportResolutionBasicTest {
         assertNotNull(buildEffective);
     }
 
-    private void addSources(final BuildAction reactor, final YangStatementSourceImpl... sources) {
+    private static void addSources(final BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index 6c4900775856cb0ea3c0e920edee12f18dd4afa5..49236cffa67e12e051a924143467559b55e1241c 100644 (file)
@@ -4,7 +4,6 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
-
 import java.util.logging.Logger;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ModelProcessingPhase;
@@ -85,7 +84,7 @@ public class IncludeResolutionTest {
 
     }
 
-    private void addSources(final BuildAction reactor, final YangStatementSourceImpl... sources) {
+    private static void addSources(final BuildAction reactor, final YangStatementSourceImpl... sources) {
         for (YangStatementSourceImpl source : sources) {
             reactor.addSource(source);
         }
index b18a2586505e3db7376d37e3d8b687e3dc2b556c..e214b2aab8895b335f0693d8486dbbd4a090415b 100644 (file)
@@ -2,19 +2,17 @@ package org.opendaylight.yangtools.yang.stmt.test;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
-
-import java.io.IOException;
-
-import java.io.FileReader;
 import java.io.BufferedReader;
 import java.io.File;
+import java.io.FileReader;
+import java.io.IOException;
+import java.net.URISyntaxException;
 import java.util.Set;
+import org.junit.Test;
 import org.opendaylight.yangtools.yang.model.api.Module;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
-import java.net.URISyntaxException;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
-import org.junit.Test;
 
 public class ModuleSourceTest {
 
@@ -38,7 +36,7 @@ public class ModuleSourceTest {
         assertEquals(readFile(moduleSourcePath), source);
     }
 
-    private String readFile(String fileName) throws IOException {
+    private static String readFile(final String fileName) throws IOException {
         BufferedReader br = new BufferedReader(new FileReader(fileName));
         try {
             StringBuilder sb = new StringBuilder();
index 0fbec5b9fc6ef2f9eb1a67ff2c380c949ef90650..8b7a08524643960a5d92c09d6ba53ca1a60becac 100644 (file)
@@ -11,23 +11,22 @@ package org.opendaylight.yangtools.yang.stmt.test;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
-
-import org.opendaylight.yangtools.yang.model.api.ModuleImport;
-import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
-import java.text.ParseException;
-import java.util.Date;
-import org.opendaylight.yangtools.yang.common.QName;
-import java.net.URI;
-import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import java.io.FileNotFoundException;
+import java.net.URI;
 import java.net.URISyntaxException;
-import org.opendaylight.yangtools.yang.model.api.SchemaContext;
+import java.text.ParseException;
+import java.util.Date;
 import java.util.Set;
 import org.junit.Test;
+import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
+import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
+import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
+import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
 import org.opendaylight.yangtools.yang.model.api.Module;
+import org.opendaylight.yangtools.yang.model.api.ModuleImport;
+import org.opendaylight.yangtools.yang.model.api.SchemaContext;
+import org.opendaylight.yangtools.yang.model.api.TypeDefinition;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
 import org.opendaylight.yangtools.yang.parser.stmt.reactor.CrossSourceStatementReactor;
@@ -154,7 +153,7 @@ public class MoreRevisionsTest {
         }
     }
 
-    private void checkContentFullTest(SchemaContext context) throws ParseException,
+    private static void checkContentFullTest(final SchemaContext context) throws ParseException,
             URISyntaxException {
 
         String yangTypesNSStr = "urn:ietf:params:xml:ns:yang:ietf-yang-types";
@@ -206,7 +205,7 @@ public class MoreRevisionsTest {
 
     }
 
-    private void checkInterfacesModuleFullTest(SchemaContext context, Date rev20100924,
+    private static void checkInterfacesModuleFullTest(final SchemaContext context, final Date rev20100924,
             final QName dateTimeTypeDef_20100924) throws URISyntaxException,
             ParseException {
         Date rev20121115 = SimpleDateFormatUtil.getRevisionFormat().parse(
@@ -223,8 +222,8 @@ public class MoreRevisionsTest {
         assertEquals(rev20100924, interfacesImport.getRevision());
     }
 
-    private void checkNetconfMonitoringModuleFullTest(SchemaContext context,
-            Date rev20130715, final QName dateTimeTypeDef_20130715)
+    private static void checkNetconfMonitoringModuleFullTest(final SchemaContext context,
+            final Date rev20130715, final QName dateTimeTypeDef_20130715)
             throws ParseException, URISyntaxException {
         Date rev20101004 = SimpleDateFormatUtil.getRevisionFormat().parse(
                 "2010-10-04");
@@ -255,7 +254,7 @@ public class MoreRevisionsTest {
         }
     }
 
-    private void checkContentSimpleTest(SchemaContext context)
+    private static void checkContentSimpleTest(final SchemaContext context)
             throws ParseException, URISyntaxException {
 
         String yangTypesNSStr = "urn:ietf:params:xml:ns:yang:ietf-yang-types";
@@ -308,8 +307,8 @@ public class MoreRevisionsTest {
 
     }
 
-    private void checkInterfacesModuleSimpleTest(SchemaContext context,
-            Date rev20100924, final QName dateTimeTypeDef_20100924)
+    private static void checkInterfacesModuleSimpleTest(final SchemaContext context,
+            final Date rev20100924, final QName dateTimeTypeDef_20100924)
             throws URISyntaxException, ParseException {
         String interfacesNSStr = "urn:ietf:params:xml:ns:yang:ietf-interfaces";
         URI interfacesNS = new URI(interfacesNSStr);
@@ -339,8 +338,8 @@ public class MoreRevisionsTest {
         assertEquals(rev20100924, interfacesImport.getRevision());
     }
 
-    private void checkNetconfMonitoringModuleSimpleTest(SchemaContext context,
-            Date rev20130715, final QName dateTimeTypeDef_20130715)
+    private static void checkNetconfMonitoringModuleSimpleTest(final SchemaContext context,
+            final Date rev20130715, final QName dateTimeTypeDef_20130715)
             throws ParseException, URISyntaxException {
         String monitoringNSStr = "urn:ietf:params:xml:ns:yang:ietf-netconf-monitoring";
         URI monitoringNS = new URI(monitoringNSStr);
@@ -371,7 +370,7 @@ public class MoreRevisionsTest {
         assertEquals(rev20130715, monitoringImport.getRevision());
     }
 
-    private boolean findTypeDef(Module module, QName typedef) {
+    private static boolean findTypeDef(final Module module, final QName typedef) {
         Set<TypeDefinition<?>> typeDefinitions = module.getTypeDefinitions();
         for (TypeDefinition<?> typeDefinition : typeDefinitions) {
             if (typeDefinition.getQName().equals(typedef)) {
index d6203a555b2c106bd30effe73b9049de99a29071..c04a49d3fccbf89325dbef4c9e74d60f4439cc9c 100644 (file)
@@ -9,9 +9,6 @@ package org.opendaylight.yangtools.yang.stmt.test;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
-
-import org.opendaylight.yangtools.yang.model.util.ExtendedType;
-
 import java.net.URI;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
@@ -19,6 +16,7 @@ import org.opendaylight.yangtools.yang.common.QNameModule;
 import org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil;
 import org.opendaylight.yangtools.yang.model.api.LeafSchemaNode;
 import org.opendaylight.yangtools.yang.model.util.BooleanType;
+import org.opendaylight.yangtools.yang.model.util.ExtendedType;
 import org.opendaylight.yangtools.yang.model.util.Int16;
 import org.opendaylight.yangtools.yang.model.util.Int32;
 import org.opendaylight.yangtools.yang.model.util.Int64;
@@ -111,7 +109,7 @@ public class YangTypes2StmtTest {
         assertEquals(BooleanType.class, lfBoolNode.getType().getClass());
     }
 
-    private void addSources(CrossSourceStatementReactor.BuildAction reactor, StatementStreamSource... sources) {
+    private static  void addSources(final CrossSourceStatementReactor.BuildAction reactor, final StatementStreamSource... sources) {
         for (StatementStreamSource source : sources) {
             reactor.addSource(source);
         }
index 86fdfaef17c92cc5519b20ceffe8bbf62785c743..83fe5b3ab99ba286a9b08ac812972d37d8c5aa7e 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.yangtools.yang.stmt.test;
 
+import static org.junit.Assert.assertNotNull;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.parser.spi.meta.ReactorException;
 import org.opendaylight.yangtools.yang.parser.spi.source.SourceException;
@@ -16,8 +17,6 @@ import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangInferencePipeline
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.YangStatementSourceImpl;
 import org.opendaylight.yangtools.yang.parser.stmt.rfc6020.effective.EffectiveSchemaContext;
 
-import static org.junit.Assert.assertNotNull;
-
 public class YangTypesStmtTest {
 
     private static final YangStatementSourceImpl TYPEFILE1 = new YangStatementSourceImpl
@@ -45,7 +44,7 @@ public class YangTypesStmtTest {
         assertNotNull(result);
     }
 
-    private void addSources(CrossSourceStatementReactor.BuildAction reactor, StatementStreamSource... sources) {
+    private static void addSources(final CrossSourceStatementReactor.BuildAction reactor, final StatementStreamSource... sources) {
         for (StatementStreamSource source : sources) {
             reactor.addSource(source);
         }