X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;ds=sidebyside;f=source%2Fnet%2Fprotocol.h;h=6d15a330295a0592b3e921d7737cde6cf515507d;hb=6625cf0e574406425c35d33110f8d99617e902df;hp=cd68d5148735024eaa77f9ef1ebe06a16064de72;hpb=ee01d294f9078fbf17dc6a0a645ad3a342c49b1a;p=libs%2Fnet.git

diff --git a/source/net/protocol.h b/source/net/protocol.h
index cd68d51..6d15a33 100644
--- a/source/net/protocol.h
+++ b/source/net/protocol.h
@@ -5,6 +5,7 @@
 #include <map>
 #include <stdexcept>
 #include <vector>
+#include <msp/core/hash.h>
 #include "receiver.h"
 
 namespace Msp {
@@ -36,9 +37,9 @@ private:
 	template<typename C>
 	struct CompoundTypeDef
 	{
-		virtual ~CompoundTypeDef() { }
+		virtual ~CompoundTypeDef() = default;
 
-		virtual std::string describe() const = 0;
+		virtual std::uint64_t get_hash() const = 0;
 		virtual char *serialize(const C &, char *, char *) const = 0;
 		virtual const char *deserialize(C &, const char *, const char *) const = 0;
 	};
@@ -50,9 +51,9 @@ private:
 
 		CompoundDef(const S &);
 
-		virtual std::string describe() const;
-		virtual char *serialize(const C &, char *, char *) const;
-		virtual const char *deserialize(C &, const char *, const char *) const;
+		std::uint64_t get_hash() const override;
+		char *serialize(const C &, char *, char *) const override;
+		const char *deserialize(C &, const char *, const char *) const override;
 	};
 
 	template<typename T>
@@ -63,7 +64,7 @@ private:
 
 		BasicSerializer(const Protocol &) { }
 
-		std::string describe() const { return get_type_signature<T>(); }
+		std::uint64_t get_hash() const { return Traits<T>::signature; }
 		char *serialize(const T &, char *, char *) const;
 		const char *deserialize(T &, const char *, const char *) const;
 	};
@@ -79,7 +80,7 @@ private:
 	public:
 		StringSerializer(const Protocol &);
 
-		std::string describe() const { return get_type_signature<std::string>(); }
+		std::uint64_t get_hash() const;
 		char *serialize(const std::string &, char *, char *) const;
 		const char *deserialize(std::string &, const char *, const char *) const;
 	};
@@ -97,7 +98,7 @@ private:
 	public:
 		ArraySerializer(const Protocol &);
 
-		std::string describe() const;
+		std::uint64_t get_hash() const;
 		char *serialize(const A &, char *, char *) const;
 		const char *deserialize(A &, const char *, const char *) const;
 	};
@@ -114,7 +115,6 @@ private:
 	public:
 		CompoundSerializer(const Protocol &);
 
-		std::string describe() const { return def.describe(); }
 		char *serialize(const C &, char *, char *) const;
 		const char *deserialize(C &, const char *, const char *) const;
 	};
@@ -138,7 +138,7 @@ private:
 	public:
 		Serializer(const Head &, Pointer, const Protocol &);
 
-		std::string describe() const;
+		std::uint64_t get_hash() const;
 		char *serialize(const P &, char *, char *) const;
 		const char *deserialize(P &, const char *, const char *) const;
 	};
@@ -153,7 +153,7 @@ private:
 			typedef Serializer<P, NullSerializer, typename Traits<N>::Serializer> Type;
 		};
 
-		std::string describe() const { return std::string(); }
+		std::uint64_t get_hash() const { return 0; }
 		char *serialize(const P &, char *b, char *) const { return b; }
 		const char *deserialize(P &, const char *b, const char *) const { return b; }
 	};
@@ -165,10 +165,11 @@ private:
 
 		PacketDefBase(unsigned);
 	public:
-		virtual ~PacketDefBase() { }
+		virtual ~PacketDefBase() = default;
+
 		virtual unsigned get_class_id() const = 0;
 		unsigned get_id() const { return id; }
-		virtual std::string describe() const = 0;
+		virtual std::uint64_t get_hash() const = 0;
 		virtual const char *dispatch(ReceiverBase &, const char *, const char *) const = 0;
 	};
 
@@ -182,17 +183,17 @@ private:
 		PacketTypeDef(unsigned);
 		~PacketTypeDef();
 
-		virtual unsigned get_class_id() const { return get_packet_class_id<P>(); }
+		unsigned get_class_id() const override { return get_packet_class_id<P>(); }
 
 		template<typename S>
 		void set_serializer(const S &);
 
 		const CompoundTypeDef<P> &get_compound() const { return *compound; }
 
-		virtual std::string describe() const { return compound->describe(); }
+		std::uint64_t get_hash() const override { return compound->get_hash(); }
 		char *serialize(const P &, char *, char *) const;
 		const char *deserialize(P &, const char *, const char *) const;
-		virtual const char *dispatch(ReceiverBase &, const char *, const char *) const;
+		const char *dispatch(ReceiverBase &, const char *, const char *) const override;
 	};
 
 	template<typename P, typename S>
@@ -212,10 +213,10 @@ private:
 
 	struct PacketHeader
 	{
-		std::uint16_t type;
-		std::uint16_t length;
+		std::uint16_t type = 0;
+		std::uint16_t length = 0;
 
-		PacketHeader();
+		PacketHeader() = default;
 		PacketHeader(std::uint16_t, std::uint16_t);
 	};
 
@@ -260,10 +261,6 @@ public:
 	std::size_t dispatch(ReceiverBase &, const char *, std::size_t) const;
 
 	std::uint64_t get_hash() const;
-
-private:
-	template<typename T>
-	static std::string get_type_signature();
 };
 
 
@@ -307,17 +304,6 @@ std::size_t Protocol::serialize(const P &pkt, char *buf, std::size_t size) const
 	return size;
 }
 
-template<typename T>
-std::string Protocol::get_type_signature()
-{
-	const std::uint16_t sig = Traits<T>::signature;
-	std::string result;
-	result += sig&0xFF;
-	if(sig>=0x100)
-		result += '0'+(sig>>8);
-	return result;
-}
-
 
 template<typename T, std::uint8_t K>
 struct Protocol::BasicTraits
@@ -366,9 +352,9 @@ Protocol::CompoundDef<C, S>::CompoundDef(const S &s):
 { }
 
 template<typename C, typename S>
-std::string Protocol::CompoundDef<C, S>::describe() const
+std::uint64_t Protocol::CompoundDef<C, S>::get_hash() const
 {
-	return "{"+serializer.describe()+"}";
+	return hash_round<64>(serializer.get_hash(), 'C');
 }
 
 template<typename C, typename S>
@@ -384,6 +370,12 @@ const char *Protocol::CompoundDef<C, S>::deserialize(C &com, const char *buf, co
 }
 
 
+inline std::uint64_t Protocol::StringSerializer::get_hash() const
+{
+	return Traits<std::string>::signature;
+}
+
+
 template<typename A>
 Protocol::ArraySerializer<A>::ArraySerializer(const Protocol &proto):
 	length_serializer(proto),
@@ -391,17 +383,17 @@ Protocol::ArraySerializer<A>::ArraySerializer(const Protocol &proto):
 { }
 
 template<typename A>
-std::string Protocol::ArraySerializer<A>::describe() const
+std::uint64_t Protocol::ArraySerializer<A>::get_hash() const
 {
-	return "["+element_serializer.describe()+"]";
+	return hash_round<64>(element_serializer.get_hash(), 'A');
 }
 
 template<typename A>
 char *Protocol::ArraySerializer<A>::serialize(const A &array, char *buf, char *end) const
 {
 	buf = length_serializer.serialize(array.size(), buf, end);
-	for(typename A::const_iterator i=array.begin(); i!=array.end(); ++i)
-		buf = element_serializer.serialize(*i, buf, end);
+	for(const auto &e: array)
+		buf = element_serializer.serialize(e, buf, end);
 	return buf;
 }
 
@@ -443,9 +435,9 @@ Protocol::Serializer<P, Head, S>::Serializer(const Head &h, Pointer p, const Pro
 { }
 
 template<typename P, typename Head, typename S>
-std::string Protocol::Serializer<P, Head, S>::describe() const
+std::uint64_t Protocol::Serializer<P, Head, S>::get_hash() const
 {
-	return Head::describe()+ser.describe();
+	return hash_update<64>(Head::get_hash(), ser.get_hash());
 }
 
 template<typename P, typename Head, typename S>