Removed print method from implementation.
Change-Id: I0bf4238e03c8b6d4aa21c6641f0c0abca855dc11
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/AutonomousSystemTlv.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/AutonomousSystemTlv.java
index f0b8120..bfe6f44 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/AutonomousSystemTlv.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/AutonomousSystemTlv.java
@@ -25,7 +25,7 @@
import com.google.common.base.MoreObjects;
/**
- * Provides Autonomous System Tlv which contains opaque value (32 Bit AS Number).
+ * Provides Autonomous-System-Tlv which contains opaque value (32 Bit AS Number).
*/
public class AutonomousSystemTlv implements PcepValueType {
@@ -47,28 +47,28 @@
private final int rawValue;
/**
- * constructor to initialize Autonomous system tlv value.
+ * Constructor to initialize rawValue.
*
- * @param rawValue value of Autonomous system tlv
+ * @param rawValue Autonomous-System-Tlv
*/
public AutonomousSystemTlv(int rawValue) {
this.rawValue = rawValue;
}
/**
- * To create instance of AutonomousSystemTlv.
+ * Returns newly created AutonomousSystemTlv object.
*
- * @param raw opaque value ofc 32 Bit AS Number
- * @return object of AutonomousSystemTlv
+ * @param raw value of opaque.
+ * @return object of Autonomous-System-Tlv
*/
public static AutonomousSystemTlv of(final int raw) {
return new AutonomousSystemTlv(raw);
}
/**
- * Returns opaque value of 32 Bit AS Number.
+ * Returns opaque value.
*
- * @return int value of rawValue
+ * @return rawValue opaque value.
*/
public int getInt() {
return rawValue;
@@ -118,22 +118,14 @@
/**
* Reads the channel buffer and returns object of AutonomousSystemTlv.
*
- * @param c type of channel buffer
- * @return object of AutonomousSystemTlv
+ * @param c input channel buffer
+ * @return object of Autonomous-System-Tlv
*/
public static AutonomousSystemTlv read(ChannelBuffer c) {
return AutonomousSystemTlv.of(c.readInt());
}
@Override
- public void print() {
- log.debug("AutonomousSystemTlv");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
- log.debug("Value: " + rawValue);
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
.add("TYPE", TYPE)
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/ErrorObjListWithOpen.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/ErrorObjListWithOpen.java
old mode 100755
new mode 100644
index df4ebbc..a183b55
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/ErrorObjListWithOpen.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/ErrorObjListWithOpen.java
@@ -13,7 +13,7 @@
import com.google.common.base.MoreObjects;
import com.google.common.base.MoreObjects.ToStringHelper;
-/*
+/**
* Provide the error object list with open object.
*/
public class ErrorObjListWithOpen {
@@ -22,14 +22,14 @@
// openObject is optional
private PcepOpenObject openObject;
// flag to check if open object is set or not
- public boolean isOpenObjectSet;
+ private boolean isOpenObjectSet;
protected static final Logger log = LoggerFactory.getLogger(ErrorObjListWithOpen.class);
- /*
- * constructor to initialize errObj,openObj.
+ /**
+ * Constructor to initialize Error and OPEN object.
*
- * @param errObj error object list
- * @param openObj open object
+ * @param errObj ERROR object list
+ * @param openObj OPEN object
*/
public ErrorObjListWithOpen(LinkedList<PcepErrorObject> errObj, PcepOpenObject openObj) {
this.llerrorObjList = errObj;
@@ -41,10 +41,10 @@
}
}
- /*
- * constructor to initialize errObj.
+ /**
+ * Constructor to initialize ERROR Object.
*
- * @param errObj error object list
+ * @param errObj ERROR Object list
*/
public ErrorObjListWithOpen(LinkedList<PcepErrorObject> errObj) {
this.llerrorObjList = errObj;
@@ -53,9 +53,9 @@
}
/**
- * Returns error type.
+ * Return list of Error Types.
*
- * @return error type
+ * @return error types list
*/
public LinkedList<Integer> getErrorType() {
LinkedList<Integer> errorType = new LinkedList<Integer>();
@@ -73,9 +73,9 @@
}
/**
- * Returns error value.
+ * Return list of Error Values.
*
- * @return error value
+ * @return error values list
*/
public LinkedList<Integer> getErrorValue() {
LinkedList<Integer> errorValue = new LinkedList<Integer>();
@@ -87,16 +87,15 @@
errorObj = errObjListIterator.next();
error = errorObj.getErrorValue();
errorValue.add(error);
-
}
}
return errorValue;
}
/**
- * Checks whether error object list is empty or not.
+ * Checks whether ERROR Object list is empty or not.
*
- * @return whether error object list is empty or not
+ * @return true if ERROR Object list is empty otherwise false
*/
public boolean isErrorObjListWithOpenPresent() {
// ( <error-obj-list> [<Open>]
@@ -107,62 +106,44 @@
/**
* Write Error Object List and Open Object to channel buffer.
*
- * @param bb of type channel buffer
+ * @param cb output channel buffer
+ * @return length of written Error object list with open
* @throws PcepParseException when mandatory fields are not set
*/
- public int write(ChannelBuffer bb) throws PcepParseException {
- int iLenStartIndex = bb.writerIndex();
+ public int write(ChannelBuffer cb) throws PcepParseException {
+ int iLenStartIndex = cb.writerIndex();
boolean bIsErrObjListFound = false;
//<error-obj-list> is mandatory , if not present throw exception.
if (llerrorObjList != null) {
ListIterator<PcepErrorObject> errObjListIterator = llerrorObjList.listIterator();
while (errObjListIterator.hasNext()) {
- errObjListIterator.next().write(bb);
+ errObjListIterator.next().write(cb);
bIsErrObjListFound = true;
}
}
if (!bIsErrObjListFound) {
- throw new PcepParseException("Error: [ErrorObjListWithOpen::write] <error-obj-list> is mandatory.");
+ throw new PcepParseException("<error-obj-list> is mandatory.");
}
//Open Object is optional , if present write.
if (openObject != null) {
- openObject.write(bb);
+ openObject.write(cb);
}
- return bb.writerIndex() - iLenStartIndex;
- }
-
- /**
- * Prints the attributes of ErrorObject List with open Object.
- */
- public void print() {
- log.debug("ErrorObjListWithOpen:");
- ListIterator<PcepErrorObject> pcepErrorObjIterator = llerrorObjList.listIterator();
- log.debug("<error-obj-list> :");
- while (pcepErrorObjIterator.hasNext()) {
- pcepErrorObjIterator.next().print();
- }
-
- log.debug("OpenObject:");
- if (openObject != null) {
- openObject.print();
- }
+ return cb.writerIndex() - iLenStartIndex;
}
@Override
public String toString() {
ToStringHelper toStrHelper = MoreObjects.toStringHelper(getClass());
- if (openObject == null) {
- toStrHelper
- .add("error Obj List", llerrorObjList);
- } else {
- toStrHelper
- .add("error Obj List", llerrorObjList)
- .add("open Object", openObject);
+ toStrHelper.add("ErrorObjList", llerrorObjList);
+
+ // OPEN Object is optional
+ if (openObject != null) {
+ toStrHelper.add("Open", openObject);
}
return toStrHelper.toString();
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv4SubObject.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv4SubObject.java
index 6ee82d4..b0576fe 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv4SubObject.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv4SubObject.java
@@ -169,21 +169,12 @@
}
@Override
- public void print() {
- log.debug("IPv4SubObject");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
- log.debug("IPv4 address: " + String.format("%08X", ipAddress));
- log.debug("Prefix Length: " + prefixLen);
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
.add("Type", TYPE)
.add("Length", LENGTH)
- .add("IPv4 Address", ipAddress)
- .add("Prefix Length", prefixLen)
+ .add("IPv4Address", ipAddress)
+ .add("PrefixLength", prefixLen)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv6SubObject.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv6SubObject.java
index 3be5e9e..ecd6132 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv6SubObject.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/IPv6SubObject.java
@@ -24,6 +24,7 @@
import org.slf4j.LoggerFactory;
import com.google.common.base.MoreObjects;
+import com.google.common.base.MoreObjects.ToStringHelper;
/**
* Provides IPv6 Sub Object.
@@ -204,25 +205,18 @@
}
@Override
- public void print() {
- log.debug("IPv6SubObject");
- log.debug("Type: ", TYPE);
- log.debug("Length: ", LENGTH);
- if (null != rawValue) {
- StringBuffer result = new StringBuffer();
- for (byte b : rawValue) {
- result.append(String.format("%02X ", b));
- }
- log.debug(result.toString());
- }
- }
-
- @Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("Type", TYPE)
- .add("Length", LENGTH)
- .add("IPv6 Address", rawValue)
- .toString();
+ ToStringHelper toStrHelper = MoreObjects.toStringHelper(getClass());
+
+ toStrHelper.add("Type", TYPE);
+ toStrHelper.add("Length", LENGTH);
+
+ StringBuffer result = new StringBuffer();
+ for (byte b : rawValue) {
+ result.append(String.format("%02X ", b));
+ }
+ toStrHelper.add("Value", result);
+
+ return toStrHelper.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PathKeySubObject.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PathKeySubObject.java
index 1a127c1..36f5c26 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PathKeySubObject.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PathKeySubObject.java
@@ -148,20 +148,12 @@
}
@Override
- public void print() {
- log.debug("PathKeySubObject");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
- log.debug("Path Key: " + pathKey);
- log.debug("PCEID: " + pceID);
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("Type", TYPE).add("Length", LENGTH)
- .add("Path Key", pathKey)
- .add("PCE ID", pceID)
+ .add("Type", TYPE)
+ .add("Length", LENGTH)
+ .add("PathKey", pathKey)
+ .add("PceID", pceID)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelDownload.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelDownload.java
old mode 100755
new mode 100644
index ebecf19..45aadd9
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelDownload.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelDownload.java
@@ -1,7 +1,6 @@
package org.onosproject.pcepio.types;
import java.util.LinkedList;
-import java.util.ListIterator;
import org.onosproject.pcepio.protocol.PcepLabelObject;
import org.onosproject.pcepio.protocol.PcepLspObject;
@@ -36,7 +35,7 @@
}
/**
- * Sets the Pcep Srp Object.
+ * Sets the Pcep SRP Object.
*
* @param srpobj PCEP SRP Object
*/
@@ -80,27 +79,9 @@
this.llLabelList = llLabelList;
}
- /**
- * Prints the attribute of PcepLableObject.
- */
- public void print() {
- log.debug("LABEL DOWNLOAD:");
- srpObject.print();
- lspObject.print();
-
- log.debug("label-list:");
- ListIterator<PcepLabelObject> listIterator = llLabelList.listIterator();
- while (listIterator.hasNext()) {
- listIterator.next().print();
- }
- }
-
@Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("SRP object", srpObject)
- .add("LSP object", lspObject)
- .add("label object list", llLabelList)
- .toString();
+ return MoreObjects.toStringHelper(getClass()).add("SrpObject", srpObject).add("LspObject", lspObject)
+ .add("LabelObjectList", llLabelList).toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelMap.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelMap.java
old mode 100755
new mode 100644
index 82be041..2d3a953
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelMap.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepLabelMap.java
@@ -92,22 +92,12 @@
this.labelObject = labelObject;
}
- /**
- * Prints the attribute of PcepLabelMap.
- */
- public void print() {
- log.debug("LABEL MAP:");
- srpObject.print();
- labelObject.print();
- fecObject.print();
- }
-
@Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("SRP object", srpObject)
- .add("Label object", labelObject)
- .add("Fec object", fecObject)
+ .add("SrpObject", srpObject)
+ .add("LabelObject", labelObject)
+ .add("FecObject", fecObject)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4Adjacency.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4Adjacency.java
index 06e99c9..df8858e 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4Adjacency.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4Adjacency.java
@@ -86,8 +86,8 @@
@Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("local IPv4 Address", localIpv4Addr)
- .add("remote IPv4 Address", remoteIpv4Addr)
+ .add("localIPv4Address", localIpv4Addr)
+ .add("remoteIPv4Address", remoteIpv4Addr)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4NodeId.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4NodeId.java
index b8d9860..dc7a7d9 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4NodeId.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv4NodeId.java
@@ -89,8 +89,6 @@
@Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("IPv4 Node Id", ipv4NodeId)
- .toString();
+ return MoreObjects.toStringHelper(getClass()).add("IPv4NodeId", ipv4NodeId).toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6Adjacency.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6Adjacency.java
index 913520a..2b1a775 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6Adjacency.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6Adjacency.java
@@ -102,8 +102,8 @@
@Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("local IPV6 address", localIpv6Addr)
- .add("remote IPV6 address", remoteIpv6Addr)
+ .add("localIPV6Address", localIpv6Addr)
+ .add("remoteIPV6Address", remoteIpv6Addr)
.toString();
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6NodeId.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6NodeId.java
index d5a5aa7..a181a34 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6NodeId.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiIpv6NodeId.java
@@ -82,8 +82,6 @@
@Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("IPV6 node ID", ipv6NodeId)
- .toString();
+ return MoreObjects.toStringHelper(getClass()).add("IPV6NodeID", ipv6NodeId).toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiUnnumberedAdjacencyIpv4.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiUnnumberedAdjacencyIpv4.java
index 30da0c8..79099b5 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiUnnumberedAdjacencyIpv4.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepNaiUnnumberedAdjacencyIpv4.java
@@ -104,10 +104,10 @@
@Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("local Node Id", localNodeId)
- .add("local Interface Id", localInterfaceId)
- .add("remote Node Id", remoteNodeId)
- .add("remote Interface Id:", remoteInterfaceId)
+ .add("localNodeId", localNodeId)
+ .add("localInterfaceId", localInterfaceId)
+ .add("remoteNodeId", remoteNodeId)
+ .add("remoteInterfaceId:", remoteInterfaceId)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepObjectHeader.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepObjectHeader.java
old mode 100755
new mode 100644
index af43a77..ab3d396
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepObjectHeader.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepObjectHeader.java
@@ -68,6 +68,7 @@
* @param bIFlag I flag
* @param objLen PCEP object length
*/
+
public PcepObjectHeader(byte objClass, byte objType, boolean bPFlag, boolean bIFlag, short objLen) {
this.objClass = objClass;
this.objType = objType;
@@ -169,12 +170,12 @@
/**
* Writes Byte stream of PCEP object header to channel buffer.
*
- * @param bb of type channel buffer
+ * @param cb output channel buffer
* @return objLenIndex object length index in channel buffer
*/
- public int write(ChannelBuffer bb) {
+ public int write(ChannelBuffer cb) {
- bb.writeByte(this.objClass);
+ cb.writeByte(this.objClass);
byte temp = (byte) (this.objType << OBJECT_TYPE_SHIFT_VALUE);
if (this.bPFlag) {
temp = (byte) (temp | PFLAG_SET);
@@ -182,55 +183,38 @@
if (this.bIFlag) {
temp = (byte) (temp | IFLAG_SET);
}
- bb.writeByte(temp);
- int objLenIndex = bb.writerIndex();
- bb.writeShort((short) 0);
+ cb.writeByte(temp);
+ int objLenIndex = cb.writerIndex();
+ cb.writeShort((short) 0);
return objLenIndex;
}
/**
* Read from channel buffer and Returns PCEP Objects header.
*
- * @param bb of type channel buffer
+ * @param cb of type channel buffer
* @return PCEP Object header
*/
- public static PcepObjectHeader read(ChannelBuffer bb) {
+ public static PcepObjectHeader read(ChannelBuffer cb) {
byte objClass;
byte objType;
boolean bPFlag;
boolean bIFlag;
short objLen;
- objClass = bb.readByte();
- byte temp = bb.readByte();
+ objClass = cb.readByte();
+ byte temp = cb.readByte();
bIFlag = ((temp & IFLAG_SET) == IFLAG_SET) ? true : false;
bPFlag = ((temp & PFLAG_SET) == PFLAG_SET) ? true : false;
objType = (byte) (temp >> OBJECT_TYPE_SHIFT_VALUE);
- objLen = bb.readShort();
+ objLen = cb.readShort();
return new PcepObjectHeader(objClass, objType, bPFlag, bIFlag, objLen);
}
- /**
- * Prints the Attributes of PCEP Object header.
- */
- public void print() {
-
- log.debug("PcepObjectHeader");
- log.debug("Object Class: " + objClass);
- log.debug("Object Type: " + objType);
- log.debug("Object Length: " + objLen);
- log.debug("P flag: " + bPFlag);
- log.debug("I flag: " + bIFlag);
- }
-
@Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("Object class:", objClass)
- .add("Object type:", objType)
- .add("Object length:", objLen)
- .add("P flag:", bPFlag)
- .add("I flag:", bIFlag)
+ return MoreObjects.toStringHelper(getClass()).add("ObjectClass", objClass).add("ObjectType", objType)
+ .add("ObjectLength", objLen).add("PFlag", (bPFlag) ? 1 : 0).add("IFlag", (bIFlag) ? 1 : 0)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpErrorSpec.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpErrorSpec.java
index fea1554..34e9e53 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpErrorSpec.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpErrorSpec.java
@@ -40,8 +40,4 @@
* @return class type
*/
byte getClassType();
-
- @Override
- String toString();
-
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv4ErrorSpec.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv4ErrorSpec.java
index 057ef6a..420392e 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv4ErrorSpec.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv4ErrorSpec.java
@@ -146,17 +146,12 @@
}
@Override
- public void print() {
- // TODO Auto-generated method stub
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("IPv4 Address:", ipv4Addr)
+ .add("IPv4Address:", ipv4Addr)
.add("flags:", flags)
- .add("error Code:", errCode)
- .add("error Value:", errValue)
+ .add("errorCode:", errCode)
+ .add("errorValue:", errValue)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv6ErrorSpec.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv6ErrorSpec.java
index 34d6049..7eade7a 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv6ErrorSpec.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpIpv6ErrorSpec.java
@@ -136,17 +136,12 @@
}
@Override
- public void print() {
- // TODO Auto-generated method stub
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("IPv6 Address:", ipv6Addr)
+ .add("IPv6Address:", ipv6Addr)
.add("flags:", flags)
- .add("error Code:", errCode)
- .add("error Value:", errValue)
+ .add("errorCode:", errCode)
+ .add("errorValue:", errValue)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpObjectHeader.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpObjectHeader.java
old mode 100755
new mode 100644
index 6de06e1..0455941
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpObjectHeader.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpObjectHeader.java
@@ -52,7 +52,7 @@
private short objLen;
/**
- * Constructor to initialize class num , length and type.
+ * Constructor to initialize class num, length and type.
*
* @param objClassNum object class number
* @param objClassType object class type
@@ -65,7 +65,7 @@
}
/**
- * Sets the Class num.
+ * Sets the Class-Num.
*
* @param value object class number
*/
@@ -121,52 +121,38 @@
/**
* Writes the byte stream of PcepRsvpObjectHeader to channel buffer.
*
- * @param bb of type channel buffer
+ * @param cb of type channel buffer
* @return object length index in channel buffer
*/
- public int write(ChannelBuffer bb) {
- int iLenStartIndex = bb.writerIndex();
- bb.writeShort((short) 0);
- bb.writeByte(this.objClassNum);
- bb.writeByte(this.objClassType);
- return bb.writerIndex() - iLenStartIndex;
+ public int write(ChannelBuffer cb) {
+ int iLenStartIndex = cb.writerIndex();
+ cb.writeShort((short) 0);
+ cb.writeByte(this.objClassNum);
+ cb.writeByte(this.objClassType);
+ return cb.writerIndex() - iLenStartIndex;
}
/**
* Reads the PcepRsvpObjectHeader.
*
- * @param bb of type channel buffer
+ * @param cb input channel buffer
* @return PcepRsvpObjectHeader
*/
- public static PcepRsvpObjectHeader read(ChannelBuffer bb) {
- log.debug("PcepRsvpObjectHeader ::read ");
+ public static PcepRsvpObjectHeader read(ChannelBuffer cb) {
+ log.debug("read ");
byte objClassNum;
byte objClassType;
short objLen;
- objLen = bb.readShort();
- objClassNum = bb.readByte();
- objClassType = bb.readByte();
+ objLen = cb.readShort();
+ objClassNum = cb.readByte();
+ objClassType = cb.readByte();
return new PcepRsvpObjectHeader(objClassNum, objClassType, objLen);
}
- /**
- * Prints the attribute of PcepRsvpObjectHeader.
- */
- public void print() {
-
- log.debug("PcepObjectHeader");
- log.debug("Object Class-Num: " + objClassNum);
- log.debug("Object C-Type: " + objClassType);
- log.debug("Object Length: " + objLen);
- }
-
@Override
public String toString() {
- return MoreObjects.toStringHelper(getClass())
- .add("Object Class-Num: " , objClassNum)
- .add("Object C-Type: " , objClassType)
- .add("Object Length: " , objLen)
- .toString();
+ return MoreObjects.toStringHelper(getClass()).add("ObjectClassNum", objClassNum)
+ .add("ObjectCType", objClassType).add("ObjectLength", objLen).toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpSpecObjHeader.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpSpecObjHeader.java
index b39bfde..9d9cf73 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpSpecObjHeader.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpSpecObjHeader.java
@@ -144,22 +144,12 @@
return new PcepRsvpSpecObjHeader(objLen, objClassNum, objClassType);
}
- /**
- * Prints the attribute of PcepRsvpObjectHeader.
- */
- public void print() {
-
- log.debug("PcepObjectHeader");
- log.debug("Object Class-Num: " + objClassNum);
- log.debug("Object C-Type: " + objClassType);
- log.debug("Object Length: " + objLen);
- }
@Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("Object Class-Num: " , objClassNum)
- .add("Object C-Type: " , objClassType)
- .add("Object Length: " , objLen)
+ .add("ObjectClassNum: " , objClassNum)
+ .add("ObjectCType: " , objClassType)
+ .add("ObjectLength: " , objLen)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpUserErrorSpec.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpUserErrorSpec.java
index 3dd036d..1a0419c 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpUserErrorSpec.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepRsvpUserErrorSpec.java
@@ -204,19 +204,14 @@
}
@Override
- public void print() {
- // TODO Auto-generated method stub
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
- .add("enterprise Number:", enterpriseNum)
- .add("sub Organization:", subOrg)
- .add("err Desc Length:", errDescLen)
- .add("user Error Value:", userErrorValue)
- .add("err Desc:", errDesc)
- .add("Rsvp User Spec Sub Object:", llRsvpUserSpecSubObj)
+ .add("enterpriseNumber:", enterpriseNum)
+ .add("subOrganization:", subOrg)
+ .add("errDescLength:", errDescLen)
+ .add("userErrorValue:", userErrorValue)
+ .add("errDesc:", errDesc)
+ .add("RsvpUserSpecSubObject:", llRsvpUserSpecSubObj)
.toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepValueType.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepValueType.java
index 9d3599b..cc23a37 100755
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepValueType.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/PcepValueType.java
@@ -52,9 +52,4 @@
* @return length of bytes written to channel buffer
*/
public int write(ChannelBuffer bb);
-
- /**
- * Prints the Attributes of PCEP Message.
- */
- public void print();
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SrEroSubObject.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SrEroSubObject.java
index 52fff7e..5fd56c4 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SrEroSubObject.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SrEroSubObject.java
@@ -308,17 +308,6 @@
}
@Override
- public void print() {
- log.debug("SrEroSubObject");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
-
- log.debug("st:" + st + " bFFlag:" + bFFlag + " bSFlag:" + bSFlag + " bCFlag:" + bCFlag + " bMFlag:" + bMFlag
- + " sID:" + sID + " nAI:" + nai);
-
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
.add("Type", TYPE)
@@ -332,5 +321,4 @@
.add("nAI", nai)
.toString();
}
-
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulIPv4LspIdentidiersTlv.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulIPv4LspIdentidiersTlv.java
index 593221c..863698f 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulIPv4LspIdentidiersTlv.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulIPv4LspIdentidiersTlv.java
@@ -198,28 +198,13 @@
}
@Override
- public void print() {
-
- log.debug("StatefulIPv4LspIdentidiersTlv");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
- log.debug("Ipv4IngressAddress: " + ipv4IngressAddress);
- log.debug("LspId: " + lspId);
- log.debug("TunnelId: " + tunnelId);
- log.debug("ExtendedTunnelId: " + extendedTunnelId);
- log.debug("Ipv4EgressAddress: " + ipv4EgressAddress);
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass())
.add("Type:", TYPE)
.add("Length:", LENGTH)
.add("Ipv4IngressAddress:", ipv4IngressAddress)
- .add("LspId:", lspId)
- .add("TunnelId:", tunnelId)
+ .add("LspId:", lspId).add("TunnelId:", tunnelId)
.add("ExtendedTunnelId:", extendedTunnelId)
- .add("Ipv4EgressAddress:", ipv4EgressAddress)
- .toString();
+ .add("Ipv4EgressAddress:", ipv4EgressAddress).toString();
}
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulLspErrorCodeTlv.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulLspErrorCodeTlv.java
index 39aa523..3239294 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulLspErrorCodeTlv.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulLspErrorCodeTlv.java
@@ -131,15 +131,6 @@
}
@Override
- public void print() {
-
- log.debug("StatefulLspErrorCodeTlv");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + LENGTH);
- log.debug("Value: " + rawValue);
- }
-
- @Override
public String toString() {
return MoreObjects.toStringHelper(getClass()).add("Type", TYPE).add("Length", LENGTH).add("Value", rawValue)
.toString();
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulRsvpErrorSpecTlv.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulRsvpErrorSpecTlv.java
index bb36465..7590033 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulRsvpErrorSpecTlv.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/StatefulRsvpErrorSpecTlv.java
@@ -206,29 +206,13 @@
}
@Override
- public void print() {
-
- log.debug("StatefulRsvpErrorSpecTlv");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + hLength);
- if (isErrSpceObjSet) {
- rsvpErrSpecObj.print();
- }
- }
-
- @Override
public String toString() {
ToStringHelper toStrHelper = MoreObjects.toStringHelper(getClass());
if (!isErrSpceObjSet) {
- toStrHelper
- .add("Type", TYPE)
- .add("Length", hLength);
+ toStrHelper.add("Type", TYPE).add("Length", hLength);
} else {
- toStrHelper
- .add("Type", TYPE)
- .add("Length", hLength)
- .add("RSVP Error Spec Object", rsvpErrSpecObj);
+ toStrHelper.add("Type", TYPE).add("Length", hLength).add("RSVPErrorSpecObject", rsvpErrSpecObj);
}
return toStrHelper.toString();
}
diff --git a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SymbolicPathNameTlv.java b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SymbolicPathNameTlv.java
index e2d36b5..12cf435 100644
--- a/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SymbolicPathNameTlv.java
+++ b/pcep/pcepio/src/main/java/org/onosproject/pcepio/types/SymbolicPathNameTlv.java
@@ -151,15 +151,7 @@
}
@Override
- public void print() {
- log.debug("SymbolicPathNameTlv");
- log.debug("Type: " + TYPE);
- log.debug("Length: " + hLength);
- log.debug("Symbolic Path Name : " + rawValue);
- }
-
- @Override
public String toString() {
- return MoreObjects.toStringHelper(getClass()).add("Symbolic Path Name ", rawValue).toString();
+ return MoreObjects.toStringHelper(getClass()).add("SymbolicPathName ", rawValue).toString();
}
}