Added more javadocs. 75/1075/1
authorMartin Vitez <mvitez@cisco.com>
Mon, 2 Sep 2013 14:38:59 +0000 (16:38 +0200)
committerMartin Vitez <mvitez@cisco.com>
Mon, 2 Sep 2013 14:38:59 +0000 (16:38 +0200)
Removed unused methods from UsesNodeBuilder.

Signed-off-by: Martin Vitez <mvitez@cisco.com>
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/api/UsesNodeBuilder.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ModuleBuilder.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/UsesNodeBuilderImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/impl/YangParserImpl.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/util/CopyUtils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/util/GroupingUtils.java
yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/util/ParserUtils.java

index a861c45f62d56d5955a145e60ffac4741756bf07..0ead00294692d2d31b1559b8cc3955587e1af9c8 100644 (file)
@@ -21,62 +21,186 @@ import org.opendaylight.yangtools.yang.parser.util.RefineHolder;
  */
 public interface UsesNodeBuilder extends GroupingMember, Builder {
 
+    /**
+     * Get parent of this uses node. Since uses can be defined only under on of
+     * module, container, list, case, grouping, input, output, notification or
+     * augment, return type is DataNodeContainerBuilder.
+     */
     DataNodeContainerBuilder getParent();
 
-    String getGroupingName();
-
+    /**
+     * Get grouping path as string.
+     *
+     * @return grouping path as String
+     */
+    String getGroupingPathAsString();
+
+    /**
+     * Get grouping path.
+     *
+     * @return grouping path as SchemaPath
+     */
     SchemaPath getGroupingPath();
 
+    /**
+     * Get grouping definition.
+     *
+     * @return GroupingDefinition if present, null otherwise
+     */
     GroupingDefinition getGroupingDefinition();
 
+    /**
+     * Set grouping definition.
+     *
+     * @param groupingDefinition
+     *            GroupingDefinition object
+     */
     void setGroupingDefinition(GroupingDefinition groupingDefinition);
 
+    /**
+     * Get grouping builder.
+     *
+     * @return GroupingBuilder if present, null otherwise
+     */
     GroupingBuilder getGroupingBuilder();
 
+    /**
+     * Set grouping builder.
+     *
+     * @param grouping
+     *            GroupingBuilder object
+     */
     void setGrouping(GroupingBuilder grouping);
 
+    /**
+     * Get augmentations defined in this uses node.
+     *
+     * @return set of augmentations defined in this node
+     */
     Set<AugmentationSchemaBuilder> getAugmentations();
 
+    /**
+     * Add augment builder.
+     *
+     * @param builder
+     *            new builder of augment statement
+     */
     void addAugment(AugmentationSchemaBuilder builder);
 
-    boolean isAugmenting();
-
-    void setAugmenting(boolean augmenting);
-
+    /**
+     * Get refine statements.
+     *
+     * @return list of RefineHolder objects
+     */
     List<RefineHolder> getRefines();
 
+    /**
+     * Get refined nodes.
+     *
+     * @return List of refined SchemaNodeBuilder objects
+     */
     List<SchemaNodeBuilder> getRefineNodes();
 
+    /**
+     * Add refine statement.
+     *
+     * @param refine
+     *            new RefineHolder object
+     */
     void addRefine(RefineHolder refine);
 
+    /**
+     * Add refine node.
+     *
+     * @param refineNode
+     *            refined DataSchemaNodeBuilder object
+     */
     void addRefineNode(DataSchemaNodeBuilder refineNode);
 
+    /**
+     * Build new UsesNode object.
+     */
     UsesNode build();
 
+    /**
+     * Get child nodes defined in target grouping.
+     *
+     * @return set of DataSchemaNodeBuilder objects
+     */
     Set<DataSchemaNodeBuilder> getTargetChildren();
 
+    /**
+     * Set reference to target grouping child nodes.
+     *
+     * @param targetChildren
+     *            set of child nodes defined in target grouping
+     */
     void setTargetChildren(Set<DataSchemaNodeBuilder> targetChildren);
 
+    /**
+     * Get groupings defined in target grouping.
+     *
+     * @return set of GroupingBuilder objects
+     */
     Set<GroupingBuilder> getTargetGroupings();
 
+    /**
+     * Set reference to target grouping groupings.
+     *
+     * @param targetGroupings
+     *            set of groupings defined in target grouping
+     */
     void setTargetGroupings(Set<GroupingBuilder> targetGroupings);
 
+    /**
+     * Get type definitions defined in target grouping.
+     *
+     * @return set of typedefs defined in target grouping
+     */
     Set<TypeDefinitionBuilder> getTargetTypedefs();
 
+    /**
+     * Set reference to target grouping typedefs.
+     *
+     * @param targetTypedefs
+     *            set of typedefs defined in target grouping
+     */
     void setTargetTypedefs(Set<TypeDefinitionBuilder> targetTypedefs);
 
+    /**
+     * Get unknown nodes defined in target grouping.
+     *
+     * @return list of unknown nodes defined in target grouping
+     */
     List<UnknownSchemaNodeBuilder> getTargetUnknownNodes();
 
+    /**
+     * Set reference to target grouping unknown nodes.
+     *
+     * @param targetUnknownNodes
+     *            list of unknown nodes defined in target grouping.
+     */
     void setTargetUnknownNodes(List<UnknownSchemaNodeBuilder> targetUnknownNodes);
 
+    /**
+     *
+     * @return true, if this object was built based on another UsesNodeBuilder,
+     *         false otherwise
+     */
     boolean isCopy();
 
+    /**
+     *
+     * @return true, if target grouping objects was loaded already, false
+     *         otherwise
+     */
     boolean isDataCollected();
 
+    /**
+     * Set if target grouping objects was loaded already.
+     *
+     * @param dataCollected
+     */
     void setDataCollected(boolean dataCollected);
 
-    boolean isParentUpdated();
-
-    void setParentUpdated(boolean parentUpdated);
-
 }
index 71ff0717b973df0c5d4d6afe190974999cb2e9b0..01ee714694d3e198988329d344e9460b2cadb701 100644 (file)
@@ -465,9 +465,6 @@ public class ModuleBuilder extends AbstractDataNodeContainerBuilder {
             if (!(parent instanceof DataNodeContainerBuilder)) {
                 throw new YangParseException(name, line, "Unresolved parent of uses '" + groupingPathStr + "'.");
             }
-            if (parent instanceof AugmentationSchemaBuilder) {
-                usesBuilder.setAugmenting(true);
-            }
             ((DataNodeContainerBuilder) parent).addUsesNode(usesBuilder);
         }
         allUsesNodes.add(usesBuilder);
index 94416471bd066dc29073f9b21dbe5e9e5a701f3c..c1978e836c79eac220e2bcb3a11401bb1f502d65 100644 (file)
@@ -41,7 +41,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
     private SchemaPath groupingPath;\r
     private GroupingDefinition groupingDefinition;\r
     private GroupingBuilder groupingBuilder;\r
-    private boolean augmenting;\r
     private boolean addedByUses;\r
     private final Set<AugmentationSchemaBuilder> addedAugments = new HashSet<AugmentationSchemaBuilder>();\r
     private final List<SchemaNodeBuilder> refineBuilders = new ArrayList<SchemaNodeBuilder>();\r
@@ -54,7 +53,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
 \r
     private final boolean isCopy;\r
     private boolean dataCollected;\r
-    private boolean parentUpdated;\r
 \r
     @Override\r
     public boolean isCopy() {\r
@@ -71,16 +69,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
         this.dataCollected = dataCollected;\r
     }\r
 \r
-    @Override\r
-    public boolean isParentUpdated() {\r
-        return parentUpdated;\r
-    }\r
-\r
-    @Override\r
-    public void setParentUpdated(boolean parentUpdated) {\r
-        this.parentUpdated = parentUpdated;\r
-    }\r
-\r
     public UsesNodeBuilderImpl(final String moduleName, final int line, final String groupingName) {\r
         super(moduleName, line);\r
         this.groupingName = groupingName;\r
@@ -97,7 +85,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
     public UsesNode build() {\r
         if (!isBuilt) {\r
             instance = new UsesNodeImpl(groupingPath);\r
-            instance.setAugmenting(augmenting);\r
             instance.setAddedByUses(addedByUses);\r
 \r
             // AUGMENTATIONS\r
@@ -136,7 +123,8 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
     @Override\r
     public void setParent(Builder parent) {\r
         if (!(parent instanceof DataNodeContainerBuilder)) {\r
-            throw new YangParseException(moduleName, line, "Unresolved parent of uses '" + groupingName + "'.");\r
+            throw new YangParseException(moduleName, line,\r
+                    "Parent of 'uses' has to be instance of DataNodeContainerBuilder, but was: '" + parent + "'.");\r
         }\r
         this.parent = (DataNodeContainerBuilder) parent;\r
     }\r
@@ -173,7 +161,7 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
     }\r
 \r
     @Override\r
-    public String getGroupingName() {\r
+    public String getGroupingPathAsString() {\r
         return groupingName;\r
     }\r
 \r
@@ -187,16 +175,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
         addedAugments.add(augmentBuilder);\r
     }\r
 \r
-    @Override\r
-    public boolean isAugmenting() {\r
-        return augmenting;\r
-    }\r
-\r
-    @Override\r
-    public void setAugmenting(final boolean augmenting) {\r
-        this.augmenting = augmenting;\r
-    }\r
-\r
     @Override\r
     public boolean isAddedByUses() {\r
         return addedByUses;\r
@@ -272,6 +250,8 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
         final int prime = 31;\r
         int result = 1;\r
         result = prime * result + ((groupingName == null) ? 0 : groupingName.hashCode());\r
+        result = prime * result + ((parent == null) ? 0 : parent.hashCode());\r
+        result = prime * result + ((refines == null) ? 0 : refines.hashCode());\r
         return result;\r
     }\r
 \r
@@ -289,7 +269,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
                 return false;\r
         } else if (!groupingName.equals(other.groupingName))\r
             return false;\r
-\r
         if (parent == null) {\r
             if (other.parent != null)\r
                 return false;\r
@@ -311,7 +290,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
     public final class UsesNodeImpl implements UsesNode {\r
         private final SchemaPath groupingPath;\r
         private Set<AugmentationSchema> augmentations = Collections.emptySet();\r
-        private boolean augmenting;\r
         private boolean addedByUses;\r
         private Map<SchemaPath, SchemaNode> refines = Collections.emptyMap();\r
         private List<UnknownSchemaNode> unknownNodes = Collections.emptyList();\r
@@ -338,11 +316,7 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
 \r
         @Override\r
         public boolean isAugmenting() {\r
-            return augmenting;\r
-        }\r
-\r
-        private void setAugmenting(final boolean augmenting) {\r
-            this.augmenting = augmenting;\r
+            return false;\r
         }\r
 \r
         @Override\r
@@ -385,7 +359,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
             int result = 1;\r
             result = prime * result + ((groupingPath == null) ? 0 : groupingPath.hashCode());\r
             result = prime * result + ((augmentations == null) ? 0 : augmentations.hashCode());\r
-            result = prime * result + (augmenting ? 1231 : 1237);\r
             return result;\r
         }\r
 \r
@@ -415,9 +388,6 @@ public final class UsesNodeBuilderImpl extends AbstractBuilder implements UsesNo
             } else if (!augmentations.equals(other.augmentations)) {\r
                 return false;\r
             }\r
-            if (augmenting != other.augmenting) {\r
-                return false;\r
-            }\r
             return true;\r
         }\r
 \r
index 7cd05c0534ae12df6e133cf64020247b50834c6c..48bca6716c79b508296680caa581e8f337eafef8 100644 (file)
@@ -669,7 +669,7 @@ public final class YangParserImpl implements YangModelParser {
             if (targetGroupingBuilder == null) {
                 if (context == null) {
                     throw new YangParseException(module.getName(), usesNode.getLine(), "Referenced grouping '"
-                            + usesNode.getGroupingName() + "' not found.");
+                            + usesNode.getGroupingPathAsString() + "' not found.");
                 } else {
                     GroupingDefinition targetGroupingDefinition = GroupingUtils.getTargetGroupingFromContext(usesNode,
                             module, context);
@@ -707,7 +707,7 @@ public final class YangParserImpl implements YangModelParser {
             }
         }
 
-        // new cycle is must because in collecting data process new uses could
+        // new loop is must because in collecting data process new uses could
         // be created
         final List<UsesNodeBuilder> allModulesUses = new ArrayList<>();
         for (Map.Entry<String, TreeMap<Date, ModuleBuilder>> entry : modules.entrySet()) {
@@ -764,7 +764,7 @@ public final class YangParserImpl implements YangModelParser {
             }
         }
 
-        // new cycle is must because in collecting data process new uses could
+        // new loop is must because in collecting data process new uses could
         // be created
         final List<UsesNodeBuilder> allModulesUses = new ArrayList<>();
         for (Map.Entry<String, TreeMap<Date, ModuleBuilder>> entry : modules.entrySet()) {
index 65c81b9e91f6e87d77896f9112c3d3d47036b3b8..971ee8293a605399cf6f5ebe7e5b92a42cb6e83e 100644 (file)
@@ -54,6 +54,9 @@ import org.opendaylight.yangtools.yang.parser.builder.impl.UsesNodeBuilderImpl;
 
 public class CopyUtils {
 
+    private CopyUtils() {
+    }
+
     /**
      * Create copy of DataSchemaNodeBuilder with new parent. If updateQName is
      * true, qname of node will be corrected based on new parent.
@@ -411,11 +414,10 @@ public class CopyUtils {
 
     static UsesNodeBuilder copyUses(UsesNodeBuilder old, Builder newParent) {
         UsesNodeBuilder copy = new UsesNodeBuilderImpl(newParent.getModuleName(), newParent.getLine(),
-                old.getGroupingName(), true);
+                old.getGroupingPathAsString(), true);
         copy.setParent(newParent);
         copy.setGroupingDefinition(old.getGroupingDefinition());
         copy.setGrouping(old.getGroupingBuilder());
-        copy.setAugmenting(old.isAugmenting());
         copy.setAddedByUses(old.isAddedByUses());
         copy.getAugmentations().addAll(old.getAugmentations());
         copy.getRefineNodes().addAll(old.getRefineNodes());
index 76795548d26e4af9d927675e30e7a34e86752e84..058d42542e019e8744b205f3c890cffba5083585 100644 (file)
@@ -7,8 +7,6 @@
  */
 package org.opendaylight.yangtools.yang.parser.util;
 
-import static org.opendaylight.yangtools.yang.parser.util.ParserUtils.createSchemaPath;
-
 import java.net.URI;
 import java.util.ArrayList;
 import java.util.Date;
@@ -45,6 +43,9 @@ import org.opendaylight.yangtools.yang.parser.builder.impl.UnknownSchemaNodeBuil
 
 public class GroupingUtils {
 
+    private GroupingUtils() {
+    }
+
     /**
      * Search given modules for grouping by name defined in uses node.
      *
@@ -59,7 +60,7 @@ public class GroupingUtils {
     public static GroupingBuilder getTargetGroupingFromModules(final UsesNodeBuilder usesBuilder,
             final Map<String, TreeMap<Date, ModuleBuilder>> modules, final ModuleBuilder module) {
         final int line = usesBuilder.getLine();
-        final String groupingString = usesBuilder.getGroupingName();
+        final String groupingString = usesBuilder.getGroupingPathAsString();
         String groupingPrefix;
         String groupingName;
 
@@ -130,7 +131,7 @@ public class GroupingUtils {
     public static GroupingDefinition getTargetGroupingFromContext(final UsesNodeBuilder usesBuilder,
             final ModuleBuilder module, final SchemaContext context) {
         final int line = usesBuilder.getLine();
-        String groupingString = usesBuilder.getGroupingName();
+        String groupingString = usesBuilder.getGroupingPathAsString();
         String groupingPrefix;
         String groupingName;
 
@@ -221,15 +222,28 @@ public class GroupingUtils {
         }
     }
 
+    /**
+     * Read data defined in target grouping builder, make a copy and add them to
+     * uses node builder.
+     *
+     * @param usesNode
+     *            uses node builder
+     */
     public static void collectUsesData(UsesNodeBuilder usesNode) {
-        usesNode.setTargetChildren(collectUsesChildNodes(usesNode));
-        usesNode.setTargetTypedefs(collectUsesTypedefs(usesNode));
-        usesNode.setTargetGroupings(collectUsesGroupings(usesNode));
-        usesNode.setTargetUnknownNodes(collectUsesUnknownNodes(usesNode));
+        usesNode.setTargetChildren(collectTargetChildNodes(usesNode));
+        usesNode.setTargetTypedefs(collectTargetTypedefs(usesNode));
+        usesNode.setTargetGroupings(collectTargetGroupings(usesNode));
+        usesNode.setTargetUnknownNodes(collectTargetUnknownNodes(usesNode));
         usesNode.setDataCollected(true);
     }
 
-    private static Set<DataSchemaNodeBuilder> collectUsesChildNodes(UsesNodeBuilder usesNode) {
+    /**
+     * Read child nodes defined in target grouping and make a copy of them.
+     *
+     * @param usesNode
+     * @return copy of child nodes defined in uses target grouping
+     */
+    private static Set<DataSchemaNodeBuilder> collectTargetChildNodes(UsesNodeBuilder usesNode) {
         final GroupingBuilder target = usesNode.getGroupingBuilder();
         Set<DataSchemaNodeBuilder> childNodes = target.getChildNodeBuilders();
         Set<DataSchemaNodeBuilder> copies = new HashSet<>();
@@ -237,12 +251,18 @@ public class GroupingUtils {
             copies.add(CopyUtils.copy(childNode, usesNode.getParent(), true));
         }
         for (UsesNodeBuilder targetUses : target.getUsesNodes()) {
-            copies.addAll(collectUsesChildNodes(targetUses));
+            copies.addAll(collectTargetChildNodes(targetUses));
         }
         return copies;
     }
 
-    private static Set<TypeDefinitionBuilder> collectUsesTypedefs(UsesNodeBuilder usesNode) {
+    /**
+     * Read typedefs defined in target grouping and make a copy of them.
+     *
+     * @param usesNode
+     * @return copy of typedefs defined in uses target grouping
+     */
+    private static Set<TypeDefinitionBuilder> collectTargetTypedefs(UsesNodeBuilder usesNode) {
         final GroupingBuilder target = usesNode.getGroupingBuilder();
         Set<TypeDefinitionBuilder> typedefs = target.getTypeDefinitionBuilders();
         Set<TypeDefinitionBuilder> copies = new HashSet<>();
@@ -250,12 +270,18 @@ public class GroupingUtils {
             copies.add(CopyUtils.copy(typedef, usesNode.getParent(), true));
         }
         for (UsesNodeBuilder targetUses : target.getUsesNodes()) {
-            copies.addAll(collectUsesTypedefs(targetUses));
+            copies.addAll(collectTargetTypedefs(targetUses));
         }
         return copies;
     }
 
-    private static Set<GroupingBuilder> collectUsesGroupings(UsesNodeBuilder usesNode) {
+    /**
+     * Read groupings defined in target grouping and make a copy of them.
+     *
+     * @param usesNode
+     * @return copy of groupings defined in uses target grouping
+     */
+    private static Set<GroupingBuilder> collectTargetGroupings(UsesNodeBuilder usesNode) {
         final GroupingBuilder target = usesNode.getGroupingBuilder();
         Set<GroupingBuilder> groupings = target.getGroupingBuilders();
         Set<GroupingBuilder> copies = new HashSet<>();
@@ -263,12 +289,18 @@ public class GroupingUtils {
             copies.add(CopyUtils.copy(grouping, usesNode.getParent(), true));
         }
         for (UsesNodeBuilder targetUses : target.getUsesNodes()) {
-            copies.addAll(collectUsesGroupings(targetUses));
+            copies.addAll(collectTargetGroupings(targetUses));
         }
         return copies;
     }
 
-    private static List<UnknownSchemaNodeBuilder> collectUsesUnknownNodes(UsesNodeBuilder usesNode) {
+    /**
+     * Read unknown nodes defined in target grouping and make a copy of them.
+     *
+     * @param usesNode
+     * @return copy of unknown nodes defined in uses target grouping
+     */
+    private static List<UnknownSchemaNodeBuilder> collectTargetUnknownNodes(UsesNodeBuilder usesNode) {
         final GroupingBuilder target = usesNode.getGroupingBuilder();
         List<UnknownSchemaNodeBuilder> unknownNodes = target.getUnknownNodeBuilders();
         List<UnknownSchemaNodeBuilder> copies = new ArrayList<>();
@@ -276,17 +308,24 @@ public class GroupingUtils {
             copies.add(CopyUtils.copy(unknownNode, usesNode.getParent(), true));
         }
         for (UsesNodeBuilder targetUses : target.getUsesNodes()) {
-            copies.addAll(collectUsesUnknownNodes(targetUses));
+            copies.addAll(collectTargetUnknownNodes(targetUses));
         }
         return copies;
     }
 
+    /**
+     * Read data defined in target grouping definition, make a copy and add them
+     * to uses node builder.
+     *
+     * @param usesNode
+     *            uses node builder
+     */
     public static void collectUsesDataFromContext(UsesNodeBuilder usesNode) {
         DataNodeContainerBuilder parent = usesNode.getParent();
         URI namespace = parent.getQName().getNamespace();
         Date revision = parent.getQName().getRevision();
         String prefix = parent.getQName().getPrefix();
-        String  moduleName = parent.getModuleName();
+        String moduleName = parent.getModuleName();
         int line = parent.getLine();
 
         // child nodes
@@ -317,7 +356,6 @@ public class GroupingUtils {
                     ((GroupingMember) newChild).setAddedByUses(true);
                 }
 
-                newChild.setPath(createSchemaPath(parent.getPath(), newQName));
                 newChildren.add(newChild);
             }
         }
@@ -329,7 +367,6 @@ public class GroupingUtils {
             QName newQName = new QName(namespace, revision, prefix, g.getQName().getLocalName());
             GroupingBuilder newGrouping = CopyUtils.createGrouping(g, newQName, moduleName, line);
             newGrouping.setAddedByUses(true);
-            newGrouping.setPath(createSchemaPath(parent.getPath(), newQName));
             newGroupings.add(newGrouping);
         }
         usesNode.setTargetGroupings(newGroupings);
@@ -340,7 +377,6 @@ public class GroupingUtils {
             QName newQName = new QName(namespace, revision, prefix, td.getQName().getLocalName());
             TypeDefinitionBuilder newType = CopyUtils.createTypedef((ExtendedType) td, newQName, moduleName, line);
             newType.setAddedByUses(true);
-            newType.setPath(createSchemaPath(parent.getPath(), newQName));
             newTypedefs.add(newType);
         }
         usesNode.setTargetTypedefs(newTypedefs);
@@ -351,7 +387,6 @@ public class GroupingUtils {
             QName newQName = new QName(namespace, revision, prefix, un.getQName().getLocalName());
             UnknownSchemaNodeBuilder newNode = CopyUtils.createUnknownSchemaNode(un, newQName, moduleName, line);
             newNode.setAddedByUses(true);
-            newNode.setPath(createSchemaPath(parent.getPath(), newQName));
             newUnknownNodes.add(newNode);
         }
         usesNode.setTargetUnknownNodes(newUnknownNodes);
@@ -359,6 +394,11 @@ public class GroupingUtils {
         usesNode.setDataCollected(true);
     }
 
+    /**
+     * Fix schema path of all nodes which were defined by this usesNode.
+     *
+     * @param usesNode
+     */
     public static void fixUsesNodesPath(UsesNodeBuilder usesNode) {
         DataNodeContainerBuilder parent = usesNode.getParent();
 
index 8928786c8cd1634a3060340bf51fcda043278363..f368e837ed54cf1cb471ded0465b575f69130dca 100644 (file)
@@ -248,6 +248,14 @@ public final class ParserUtils {
         }
     }
 
+    /**
+     * Create new schema path of node based on parent node schema path.
+     *
+     * @param node
+     *            node to correct
+     * @param parentSchemaPath
+     *            schema path of node parent
+     */
     static void correctNodePath(final SchemaNodeBuilder node, final SchemaPath parentSchemaPath) {
         // set correct path
         List<QName> targetNodePath = new ArrayList<QName>(parentSchemaPath.getPath());
@@ -279,7 +287,7 @@ public final class ParserUtils {
      *            parent of first node in path
      * @param path
      *            path to augment target
-     * @return true if augment process succeed, false otherwise
+     * @return true if augmentation process succeed, false otherwise
      */
     public static boolean processAugmentation(final AugmentationSchemaBuilder augment, final Builder firstNodeParent,
             final List<QName> path) {
@@ -344,6 +352,15 @@ public final class ParserUtils {
         return true;
     }
 
+    /**
+     * Find node with given name in uses target.
+     *
+     * @param localName
+     *            name of node to find
+     * @param uses
+     *            uses node which target grouping should be searched
+     * @return node with given name if found, null otherwise
+     */
     private static DataSchemaNodeBuilder findNodeInUses(String localName, UsesNodeBuilder uses) {
         GroupingBuilder target = uses.getGroupingBuilder();
         for (DataSchemaNodeBuilder child : target.getChildNodeBuilders()) {
@@ -497,8 +514,8 @@ public final class ParserUtils {
      * @return builder of module where this node is defined
      */
     public static ModuleBuilder getParentModule(Builder node) {
-        if(node instanceof ModuleBuilder) {
-            return (ModuleBuilder)node;
+        if (node instanceof ModuleBuilder) {
+            return (ModuleBuilder) node;
         }
 
         Builder parent = node.getParent();