diff options
author | Nathan Binkert <binkertn@umich.edu> | 2004-11-13 16:33:16 -0500 |
---|---|---|
committer | Nathan Binkert <binkertn@umich.edu> | 2004-11-13 16:33:16 -0500 |
commit | acb98fb0f64a3b6a4f2f3151b0d4fb14fdfcf7cf (patch) | |
tree | 4daefdaf22007eb42f30043334447de27e8aa7cb /dev/etherlink.cc | |
parent | bd3e3c02300312ae8d00ae9abf3fa009a2cddbc7 (diff) | |
download | gem5-acb98fb0f64a3b6a4f2f3151b0d4fb14fdfcf7cf.tar.xz |
we shouldn't ever pass around references to PacketPtrs,
const references are ok, or pass by value.
--HG--
extra : convert_revision : 7280a1c7d22b9294fddbe50f02f6f4c6ca9b2e5b
Diffstat (limited to 'dev/etherlink.cc')
-rw-r--r-- | dev/etherlink.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/dev/etherlink.cc b/dev/etherlink.cc index ccb18d363..a361713cd 100644 --- a/dev/etherlink.cc +++ b/dev/etherlink.cc @@ -105,7 +105,7 @@ EtherLink::unserialize(Checkpoint *cp, const string §ion) } void -EtherLink::Link::txComplete(PacketPtr &packet) +EtherLink::Link::txComplete(PacketPtr packet) { DPRINTF(Ethernet, "packet received: len=%d\n", packet->length); DDUMP(EthernetData, packet->data, packet->length); @@ -122,7 +122,7 @@ class LinkDelayEvent : public Event LinkDelayEvent(EtherLink::Link *link); public: - LinkDelayEvent(EtherLink::Link *link, PacketPtr &pkt, Tick when); + LinkDelayEvent(EtherLink::Link *link, PacketPtr pkt, Tick when); void process(); @@ -153,7 +153,7 @@ EtherLink::Link::txDone() } bool -EtherLink::Link::transmit(PacketPtr &pkt) +EtherLink::Link::transmit(PacketPtr pkt) { if (busy()) { DPRINTF(Ethernet, "packet not sent, link busy\n"); @@ -217,7 +217,7 @@ LinkDelayEvent::LinkDelayEvent(EtherLink::Link *l) setFlags(AutoDelete); } -LinkDelayEvent::LinkDelayEvent(EtherLink::Link *l, PacketPtr &p, Tick when) +LinkDelayEvent::LinkDelayEvent(EtherLink::Link *l, PacketPtr p, Tick when) : Event(&mainEventQueue), link(l), packet(p) { setFlags(AutoSerialize); |