X-Git-Url: http://git.tdb.fi/?p=libs%2Fnet.git;a=blobdiff_plain;f=source%2Fnet%2Fprotocol.h;fp=source%2Fnet%2Fprotocol.h;h=f040a69ef1293665b3f38c79387706c04bf75e05;hp=35d3cb2cbfe177de4214816d70643f9ff97e32bd;hb=ede42d5bb352841e2e425972e12b8ef31ddf2123;hpb=1565dec0f4a86e4bda7095dd7415234f7c7a84e9

diff --git a/source/net/protocol.h b/source/net/protocol.h
index 35d3cb2..f040a69 100644
--- a/source/net/protocol.h
+++ b/source/net/protocol.h
@@ -3,6 +3,7 @@
 
 #include <cstdint>
 #include <map>
+#include <memory>
 #include <stdexcept>
 #include <vector>
 #include <msp/core/hash.h>
@@ -142,11 +143,10 @@ private:
 	class PacketTypeDef: public PacketDefBase
 	{
 	private:
-		Serializer<P> *serializer;
+		std::unique_ptr<Serializer<P>> serializer;
 
 	public:
 		PacketTypeDef(unsigned);
-		~PacketTypeDef();
 
 		unsigned get_class_id() const override { return get_packet_class_id<P>(); }
 
@@ -188,17 +188,13 @@ private:
 		PacketHeader(std::uint16_t, std::uint16_t);
 	};
 
-	typedef std::map<unsigned, PacketDefBase *> PacketMap;
-
 	PacketTypeDef<PacketHeader> header_def;
 	unsigned next_packet_id;
-	PacketMap packet_class_defs;
-	PacketMap packet_id_defs;
+	std::map<unsigned, std::unique_ptr<PacketDefBase>> packet_class_defs;
+	std::map<unsigned, PacketDefBase *> packet_id_defs;
 
 protected:
 	Protocol(unsigned = 1);
-public:
-	~Protocol();
 
 private:
 	static unsigned get_next_packet_class_id();
@@ -206,7 +202,7 @@ private:
 	template<typename P>
 	static unsigned get_packet_class_id();
 
-	void add_packet(PacketDefBase *);
+	void add_packet(std::unique_ptr<PacketDefBase>);
 
 protected:
 	template<typename P>
@@ -248,9 +244,10 @@ unsigned Protocol::get_packet_class_id()
 template<typename P>
 Protocol::PacketDefBuilder<P, Protocol::Serializer<P>> Protocol::add(unsigned id)
 {
-	PacketTypeDef<P> *pdef = new PacketTypeDef<P>(id);
-	add_packet(pdef);
-	return PacketDefBuilder<P, Serializer<P>>(*this, *pdef, Serializer<P>());
+	std::unique_ptr<PacketTypeDef<P>> pdef = std::make_unique<PacketTypeDef<P>>(id);
+	PacketDefBuilder<P, Serializer<P>> next(*this, *pdef, Serializer<P>());
+	add_packet(move(pdef));
+	return next;
 }
 
 template<typename P>
@@ -413,21 +410,14 @@ const char *Protocol::FieldSerializer<C, Head, T>::deserialize(C &com, const cha
 template<typename P>
 Protocol::PacketTypeDef<P>::PacketTypeDef(unsigned i):
 	PacketDefBase(i),
-	serializer(new Serializer<P>)
+	serializer(std::make_unique<Serializer<P>>())
 { }
 
-template<typename P>
-Protocol::PacketTypeDef<P>::~PacketTypeDef()
-{
-	delete serializer;
-}
-
 template<typename P>
 template<typename S>
 void Protocol::PacketTypeDef<P>::set_serializer(const S &ser)
 {
-	delete serializer;
-	serializer = new S(ser);
+	serializer = std::make_unique<S>(ser);
 }
 
 template<typename P>