commit 83e3c1f7b5f1d5557987f137bba763a938c3702a
author Sho SHIMIZU <shimizu.sho@jp.fujitsu.com> Fri Jun 13 15:57:26 2014 -0700
committer Gerrit Code Review <gerrit2@gerrit.onos.onlab.us> Fri Jun 20 09:13:57 2014 -0700
tree 117aa7a21e2ef703005e3cf1a4520ffe4ba26c65
parent 91a8f502ff9b17bb2f36a8f7d36a83d088eafe03

Use Dpid class to represent DPID instead of Long for type safety

- Signatures of public methods are not changed
- Only private fields are changed to keep compatibility

Change-Id: I1d54fa54ede9b09439944cfb5f53564ff22ac339

src/main/java/net/onrc/onos/core/topology/SwitchEvent.java

diff --git a/src/main/java/net/onrc/onos/core/topology/SwitchEvent.java b/src/main/java/net/onrc/onos/core/topology/SwitchEvent.java
index 36cf8cf..16475f0 100644
--- a/src/main/java/net/onrc/onos/core/topology/SwitchEvent.java
+++ b/src/main/java/net/onrc/onos/core/topology/SwitchEvent.java
@@ -1,5 +1,7 @@
 package net.onrc.onos.core.topology;
 
+import net.onrc.onos.core.util.Dpid;
+
 import java.nio.ByteBuffer;
 
 /**
@@ -8,7 +10,7 @@
  * TODO: We probably want common base class/interface for Self-Contained Event Object.
  */
 public class SwitchEvent {
-    protected final Long dpid;
+    protected final Dpid dpid;
 
     /**
      * Default constructor for Serializer to use.
@@ -19,16 +21,16 @@
     }
 
     public SwitchEvent(Long dpid) {
-        this.dpid = dpid;
+        this.dpid = new Dpid(dpid);
     }
 
     public Long getDpid() {
-        return dpid;
+        return dpid.value();
     }
 
     @Override
     public String toString() {
-        return "[SwitchEvent 0x" + Long.toHexString(dpid) + "]";
+        return "[SwitchEvent 0x" + Long.toHexString(dpid.value()) + "]";
     }
 
     public static final int SWITCHID_BYTES = 2 + 8;
@@ -41,10 +43,10 @@
     }
 
     public byte[] getID() {
-        return getSwitchID(dpid).array();
+        return getSwitchID(dpid.value()).array();
     }
 
     public ByteBuffer getIDasByteBuffer() {
-        return getSwitchID(dpid);
+        return getSwitchID(dpid.value());
     }
 }