commit | 4785eec52935b19880c7cfae7bbbb5efad12b7b5 | [log] [tgz] |
---|---|---|
author | alshabib <alshabibi.ali@gmail.com> | Thu Dec 04 16:45:45 2014 -0800 |
committer | Gerrit Code Review <gerrit@onlab.us> | Fri Dec 05 00:59:35 2014 +0000 |
tree | 62bb45effaa821d8226cb5dadb421718872c667f | |
parent | a1a00c247e5ff9f91430afbea87da861e40f026c [diff] [blame] |
removing some TODOs Change-Id: Ic98c811573a1b37813993eedddec512ec85729f3
diff --git a/utils/misc/src/main/java/org/onlab/packet/BasePacket.java b/utils/misc/src/main/java/org/onlab/packet/BasePacket.java index 46e3c7e..4aece66 100644 --- a/utils/misc/src/main/java/org/onlab/packet/BasePacket.java +++ b/utils/misc/src/main/java/org/onlab/packet/BasePacket.java
@@ -118,9 +118,7 @@ } catch (final Exception e) { throw new RuntimeException("Could not clone packet"); } - // TODO: we are using serialize()/deserialize() to perform the - // cloning. Not the most efficient way but simple. We can revisit - // if we hit performance problems. + final byte[] data = this.serialize(); pkt.deserialize(this.serialize(), 0, data.length); pkt.setParent(this.parent);