Use std::unique_ptr for owning pointers

[libs/net.git] / source / net / protocol.cpp

diff --git a/source/net/protocol.cpp b/source/net/protocol.cpp
index ce200a2c0ea49afed1f320b0914c252a094669bd..402b84a0633cdeb4dbe45c463e2f9676af911709 100644 (file)
--- a/source/net/protocol.cpp
+++ b/source/net/protocol.cpp
@@ -18,29 +18,20 @@ Protocol::Protocol(unsigned npi):
                .fields(&PacketHeader::type, &PacketHeader::length);
 }
 
-Protocol::~Protocol()
-{
-       for(auto &kvp: packet_class_defs)
-               delete kvp.second;
-}
-
 unsigned Protocol::get_next_packet_class_id()
 {
        static unsigned next_id = 1;
        return next_id++;
 }
 
-void Protocol::add_packet(PacketDefBase *pdef)
+void Protocol::add_packet(unique_ptr<PacketDefBase> pdef)
 {
-       PacketDefBase *&ptr = packet_class_defs[pdef->get_class_id()];
+       unique_ptr<PacketDefBase> &ptr = packet_class_defs[pdef->get_class_id()];
        if(ptr)
-       {
                packet_id_defs.erase(ptr->get_id());
-               delete ptr;
-       }
-       ptr = pdef;
-       if(unsigned id = pdef->get_id())
-               packet_id_defs[id] = pdef;
+       ptr = move(pdef);
+       if(unsigned id = ptr->get_id())
+               packet_id_defs[id] = ptr.get();
 }
 
 const Protocol::PacketDefBase &Protocol::get_packet_by_class_id(unsigned id) const