Yang parser refactoring.

[yangtools.git] / yang / yang-parser-impl / src / main / java / org / opendaylight / yangtools / yang / parser / builder / impl / ExtensionBuilder.java

diff --git a/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ExtensionBuilder.java b/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ExtensionBuilder.java
index 401d9d5ecefe0cf156a0b2ca585ac12e30089f7b..1f71637cd59d24e96fe7bc5ef5491b77b3c4c2a2 100644 (file)
--- a/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ExtensionBuilder.java
+++ b/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/ExtensionBuilder.java
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.yangtools.yang.parser.builder.impl;
 
+import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
@@ -15,7 +16,6 @@ import org.opendaylight.yangtools.yang.model.api.ExtensionDefinition;
 import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 import org.opendaylight.yangtools.yang.model.api.Status;
 import org.opendaylight.yangtools.yang.model.api.UnknownSchemaNode;
-import org.opendaylight.yangtools.yang.model.api.YangNode;
 import org.opendaylight.yangtools.yang.parser.builder.api.AbstractSchemaNodeBuilder;
 import org.opendaylight.yangtools.yang.parser.util.Comparators;
 
@@ -23,25 +23,21 @@ public final class ExtensionBuilder extends AbstractSchemaNodeBuilder {
     private boolean isBuilt;
     private final ExtensionDefinitionImpl instance;
 
-    ExtensionBuilder(final String moduleName, final int line, final QName qname) {
+    ExtensionBuilder(final String moduleName, final int line, final QName qname, final SchemaPath path) {
         super(moduleName, line, qname);
-        instance = new ExtensionDefinitionImpl(qname);
+        this.schemaPath = path;
+        instance = new ExtensionDefinitionImpl(qname, path);
     }
 
     @Override
-    public ExtensionDefinition build(YangNode parent) {
+    public ExtensionDefinition build() {
         if (!isBuilt) {
-            instance.setPath(schemaPath);
-            instance.setDescription(description);
-            instance.setReference(reference);
-            instance.setStatus(status);
-
             // UNKNOWN NODES
             for (UnknownSchemaNodeBuilder un : addedUnknownNodes) {
-                unknownNodes.add(un.build(null));
+                unknownNodes.add(un.build());
             }
             Collections.sort(unknownNodes, Comparators.SCHEMA_NODE_COMP);
-            instance.setUnknownSchemaNodes(unknownNodes);
+            instance.addUnknownSchemaNodes(unknownNodes);
 
             isBuilt = true;
         }
@@ -49,12 +45,49 @@ public final class ExtensionBuilder extends AbstractSchemaNodeBuilder {
         return instance;
     }
 
+    @Override
+    public SchemaPath getPath() {
+        return instance.schemaPath;
+    }
+
+    @Override
+    public String getDescription() {
+        return instance.description;
+    }
+
+    @Override
+    public void setDescription(final String description) {
+        instance.description = description;
+    }
+
+    @Override
+    public String getReference() {
+        return instance.reference;
+    }
+
+    @Override
+    public void setReference(final String reference) {
+        instance.reference = reference;
+    }
+
+    @Override
+    public Status getStatus() {
+        return instance.status;
+    }
+
+    @Override
+    public void setStatus(Status status) {
+        if (status != null) {
+            instance.status = status;
+        }
+    }
+
     public void setYinElement(boolean yin) {
-        instance.setYinElement(yin);
+        instance.yin = yin;
     }
 
     public void setArgument(String argument) {
-        instance.setArgument(argument);
+        instance.argument = argument;
     }
 
     @Override
@@ -65,15 +98,16 @@ public final class ExtensionBuilder extends AbstractSchemaNodeBuilder {
     private final class ExtensionDefinitionImpl implements ExtensionDefinition {
         private final QName qname;
         private String argument;
-        private SchemaPath schemaPath;
+        private final SchemaPath schemaPath;
         private String description;
         private String reference;
         private Status status = Status.CURRENT;
-        private List<UnknownSchemaNode> unknownNodes = Collections.emptyList();
+        private final List<UnknownSchemaNode> unknownNodes = new ArrayList<>();
         private boolean yin;
 
-        private ExtensionDefinitionImpl(QName qname) {
+        private ExtensionDefinitionImpl(QName qname, SchemaPath path) {
             this.qname = qname;
+            this.schemaPath = path;
         }
 
         @Override
@@ -86,47 +120,29 @@ public final class ExtensionBuilder extends AbstractSchemaNodeBuilder {
             return schemaPath;
         }
 
-        private void setPath(SchemaPath schemaPath) {
-            this.schemaPath = schemaPath;
-        }
-
         @Override
         public String getDescription() {
             return description;
         }
 
-        private void setDescription(String description) {
-            this.description = description;
-        }
-
         @Override
         public String getReference() {
             return reference;
         }
 
-        private void setReference(String reference) {
-            this.reference = reference;
-        }
-
         @Override
         public Status getStatus() {
             return status;
         }
 
-        private void setStatus(Status status) {
-            if (status != null) {
-                this.status = status;
-            }
-        }
-
         @Override
         public List<UnknownSchemaNode> getUnknownSchemaNodes() {
-            return unknownNodes;
+            return Collections.unmodifiableList(unknownNodes);
         }
 
-        private void setUnknownSchemaNodes(List<UnknownSchemaNode> unknownNodes) {
+        private void addUnknownSchemaNodes(List<UnknownSchemaNode> unknownNodes) {
             if (unknownNodes != null) {
-                this.unknownNodes = unknownNodes;
+                this.unknownNodes.addAll(unknownNodes);
             }
         }
 
@@ -135,19 +151,11 @@ public final class ExtensionBuilder extends AbstractSchemaNodeBuilder {
             return argument;
         }
 
-        private void setArgument(String argument) {
-            this.argument = argument;
-        }
-
         @Override
         public boolean isYinElement() {
             return yin;
         }
 
-        private void setYinElement(boolean yin) {
-            this.yin = yin;
-        }
-
         @Override
         public int hashCode() {
             final int prime = 31;