commit e79b0ef7717aa8f64e63d4759bcac11947a506ce
author Thejaswi N K <thejaswik@huawei.com> Sat Nov 21 20:59:04 2015 +0530
committer Gerrit Code Review <gerrit@onlab.us> Sat Nov 21 17:52:08 2015 +0000
tree aa3cae342ac8e3a7e0c221b5ab976d5ee7eadb9e
parent 68d434a0e6228588eaa059934d7df5567d277daa

[onos-2603] - Implement LinkState tag attributes

Change-Id: I674197405a57a8a45a2939e364c59c4565ad756f

bgp/bgpio/src/main/java/org/onosproject/bgpio/types/attr/BgpPrefixAttrRouteTag.java

diff --git a/bgp/bgpio/src/main/java/org/onosproject/bgpio/types/attr/BgpPrefixAttrRouteTag.java b/bgp/bgpio/src/main/java/org/onosproject/bgpio/types/attr/BgpPrefixAttrRouteTag.java
index 0cf0238..426eb27 100755
--- a/bgp/bgpio/src/main/java/org/onosproject/bgpio/types/attr/BgpPrefixAttrRouteTag.java
+++ b/bgp/bgpio/src/main/java/org/onosproject/bgpio/types/attr/BgpPrefixAttrRouteTag.java
@@ -15,7 +15,8 @@
  */
 package org.onosproject.bgpio.types.attr;
 
-import java.util.Arrays;
+import java.util.ArrayList;
+import java.util.List;
 import java.util.Objects;
 
 import org.jboss.netty.buffer.ChannelBuffer;
@@ -36,18 +37,28 @@
     protected static final Logger log = LoggerFactory
             .getLogger(BgpPrefixAttrRouteTag.class);
 
-    public static final int ATTR_PREFIX_ROUTETAG = 1153;
+    public static final short ATTR_PREFIX_ROUTETAG = 1153;
 
     /* Prefix Route Tag */
-    private int[] pfxRouteTag;
+    private List<Integer> pfxRouteTag = new ArrayList<Integer>();
 
     /**
      * Constructor to initialize the values.
      *
      * @param pfxRouteTag prefix route tag
      */
-    BgpPrefixAttrRouteTag(int[] pfxRouteTag) {
-        this.pfxRouteTag = Arrays.copyOf(pfxRouteTag, pfxRouteTag.length);
+    public BgpPrefixAttrRouteTag(List<Integer> pfxRouteTag) {
+        this.pfxRouteTag = pfxRouteTag;
+    }
+
+    /**
+     * Returns object of this class with specified values.
+     *
+     * @param pfxRouteTag Prefix Metric
+     * @return object of BgpPrefixAttrRouteTag
+     */
+    public static BgpPrefixAttrRouteTag of(ArrayList<Integer> pfxRouteTag) {
+        return new BgpPrefixAttrRouteTag(pfxRouteTag);
     }
 
     /**
@@ -59,7 +70,8 @@
      */
     public static BgpPrefixAttrRouteTag read(ChannelBuffer cb)
             throws BGPParseException {
-        int[] pfxRouteTag;
+        int tmp;
+        ArrayList<Integer> pfxRouteTag = new ArrayList<Integer>();
 
         short lsAttrLength = cb.readShort();
         int len = lsAttrLength / Integer.SIZE;
@@ -70,13 +82,12 @@
                                    lsAttrLength);
         }
 
-        pfxRouteTag = new int[lsAttrLength];
-
         for (int i = 0; i < len; i++) {
-            pfxRouteTag[i] = cb.readInt();
+            tmp = cb.readInt();
+            pfxRouteTag.add(new Integer(tmp));
         }
 
-        return new BgpPrefixAttrRouteTag(pfxRouteTag);
+        return BgpPrefixAttrRouteTag.of(pfxRouteTag);
     }
 
     /**
@@ -84,7 +95,7 @@
      *
      * @return route tag
      */
-    int[] getPfxRouteTag() {
+    public List<Integer> getPfxRouteTag() {
         return pfxRouteTag;
     }