Use default member initializers where possible

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

diff --git a/source/net/protocol.h b/source/net/protocol.h
index cd68d5148735024eaa77f9ef1ebe06a16064de72..6d15a330295a0592b3e921d7737cde6cf515507d 100644 (file)
--- 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>