unsigned PortNumber
PortNumber class update
- internal representation now uses int
- toString() now converts the value as unsigned integer
- renamed value() to shortValue()
- added value() method returning port number as unsigned long
- added constructor from String
- JSON serialization now uses the new value(), so that special handling for
unsigned integer is not required
Update classes using PortNumber
- Replaced value() calls with new value(), when the return value was
converted to long immediately.
- toString() result may change if the object contained PortNumber.
- e.g., FlowEntryAction, FlowEntry, SwitchPort, ...
Change-Id: I0b1f8aa92208e165b6113c8dd6954076c0513167
diff --git a/src/main/java/net/onrc/onos/apps/forwarding/Forwarding.java b/src/main/java/net/onrc/onos/apps/forwarding/Forwarding.java
index 969b2b7..a2a8e48 100644
--- a/src/main/java/net/onrc/onos/apps/forwarding/Forwarding.java
+++ b/src/main/java/net/onrc/onos/apps/forwarding/Forwarding.java
@@ -331,7 +331,7 @@
//This code assumes the host has only one port. It should be problem.
Port destinationPort = ports.next();
- short destinationPortNum = destinationPort.getNumber().value();
+ short destinationPortNum = destinationPort.getNumber().shortValue();
Switch destinationSw = destinationPort.getSwitch();
long destinationDpid = destinationSw.getDpid().value();
@@ -525,7 +525,7 @@
outPort = (short) spfIntent.getDstPortNumber();
log.debug("Path is empty. Maybe hosts on the same switch. outPort {}", outPort);
} else {
- outPort = graphPath.get(0).getSrc().getPortNumber().value();
+ outPort = graphPath.get(0).getSrc().getPortNumber().shortValue();
log.debug("path{}, outPort {}", graphPath, outPort);
}