commit fb564502676f7f6d86b167abde4c2b57393319bf
author Yuta HIGUCHI <y-higuchi@onlab.us> Mon Jun 16 21:29:00 2014 -0700
committer Gerrit Code Review <gerrit2@gerrit.onos.onlab.us> Wed Jun 25 16:46:22 2014 -0700
tree 17e7fbc11a0abf7ff21f86023a3907ff4d68137e
parent d83844d039b6c2a134096aed2d6ef52e9e4343ab

Rename Port to PortNumber

Current util.Port actually only represent a port number.
Renaming it to PortNumber, so that it matches what it actually reprsent.

- Updated javadoc
- Removed unnecessary hashCode calculation
- Hide default constructor from public

Change-Id: Ib26c746eaa27464356c4ca0abff6af5e321afc35

src/main/java/net/onrc/onos/core/intent/ForwardAction.java

diff --git a/src/main/java/net/onrc/onos/core/intent/ForwardAction.java b/src/main/java/net/onrc/onos/core/intent/ForwardAction.java
index 373d13c..ef6dd00 100644
--- a/src/main/java/net/onrc/onos/core/intent/ForwardAction.java
+++ b/src/main/java/net/onrc/onos/core/intent/ForwardAction.java
@@ -1,6 +1,7 @@
 package net.onrc.onos.core.intent;
 
 import net.onrc.onos.core.util.FlowEntryAction;
+import net.onrc.onos.core.util.PortNumber;
 
 /**
  * A class to represent the OpenFlow forwarding action.
@@ -23,6 +24,7 @@
      *
      * @return the destination port as a String
      */
+    @Override
     public String toString() {
         return Long.toString(dstPort);
     }
@@ -35,7 +37,7 @@
     @Override
     public FlowEntryAction getFlowEntryAction() {
         FlowEntryAction action = new FlowEntryAction();
-        action.setActionOutput(new net.onrc.onos.core.util.Port((short) dstPort));
+        action.setActionOutput(new PortNumber((short) dstPort));
         return action;
     }
 
@@ -44,6 +46,7 @@
      *
      * @return hashcode
      */
+    @Override
     public int hashCode() {
         return (int) dstPort;
     }
@@ -54,6 +57,7 @@
      * @param o another object to compare to this
      * @return true if equal, false otherwise
      */
+    @Override
     public boolean equals(Object o) {
         if (!(o instanceof ForwardAction)) {
             return false;