Add move semantics to remaining HTTP Parser heirarchy

Destructor is requied because this hierarchy contains virtuals, which in turn
means the compiler will not add move constructor by default. So we must add
teh default ones in ourselves.

  Detected by Coverity Scan. Issues 1364733 and 1364736.

diff --git a/src/http/one/Parser.h b/src/http/one/Parser.h
index c467f471a0a80fd2b9c341f0dad4788663930499..6112b59d1cb4013846dce81f621fc4bae04a1ccd 100644 (file)
--- a/src/http/one/Parser.h
+++ b/src/http/one/Parser.h
@@ -41,6 +41,11 @@ class Parser : public RefCountable
 public:
     typedef SBuf::size_type size_type;
 
+    Parser() = default;
+    Parser(const Parser &) = default;
+    Parser &operator =(const Parser &) = default;
+    Parser(Parser &&) = default;
+    Parser &operator =(Parser &&) = default;
     virtual ~Parser() {}
 
     /// Set this parser back to a default state.

diff --git a/src/http/one/ResponseParser.h b/src/http/one/ResponseParser.h
index d1ef0897f9ded96020f6e67d12c97fd1e1ebc6cc..46362a0c0df203e5e31c4f043779128fba08f490 100644 (file)
--- a/src/http/one/ResponseParser.h
+++ b/src/http/one/ResponseParser.h
@@ -29,6 +29,11 @@ namespace One {
 class ResponseParser : public Http1::Parser
 {
 public:
+    ResponseParser() = default;
+    ResponseParser(const ResponseParser &) = default;
+    ResponseParser &operator =(const ResponseParser &) = default;
+    ResponseParser(ResponseParser &&) = default;
+    ResponseParser &operator =(ResponseParser &&) = default;
     virtual ~ResponseParser() {}
 
     /* Http::One::Parser API */