Remove Handshake_Message::deserialize which was an unnecessary hook.

Instead deserialize directly in the constructors that are passed the
raw message data. This makes it easier to pass contextual information
needed for decoding (eg, version numbers) where necessary.

1 files changed, 7 insertions, 7 deletions

diff --git a/src/tls/c_hello.cpp b/src/tls/c_hello.cpp
index 6c4964fb1..60f6de487 100644
--- a/src/tls/c_hello.cpp
+++ b/src/tls/c_hello.cpp
@@ -56,20 +56,20 @@ Hello_Request::Hello_Request(Record_Writer& writer)
    }
 
 /*
-* Serialize a Hello Request message
+* Deserialize a Hello Request message
 */
-MemoryVector<byte> Hello_Request::serialize() const
+Hello_Request::Hello_Request(const MemoryRegion<byte>& buf)
    {
-   return MemoryVector<byte>();
+   if(buf.size())
+      throw Decoding_Error("Hello_Request: Must be empty, and is not");
    }
 
 /*
-* Deserialize a Hello Request message
+* Serialize a Hello Request message
 */
-void Hello_Request::deserialize(const MemoryRegion<byte>& buf)
+MemoryVector<byte> Hello_Request::serialize() const
    {
-   if(buf.size())
-      throw Decoding_Error("Hello_Request: Must be empty, and is not");
+   return MemoryVector<byte>();
    }
 
 /*