Restore copy-construct on Parser hierarchy

While it does not make sense to clone a Parser and be running two
independently on the same buffer simultaneously it is also no worth the
added compexity of forbidding it at this time.

diff --git a/src/http/one/Parser.h b/src/http/one/Parser.h
index e4ec3f8471010b2da83f209f1aa8e191f265fee1..e6ae3f7259944c6ddd3ab7bc7c459ff2aa58c41a 100644 (file)
--- a/src/http/one/Parser.h
+++ b/src/http/one/Parser.h
@@ -26,14 +26,10 @@ enum ParseState {
  */
 class Parser : public RefCountable
 {
-    explicit Parser(const Parser&); // do not implement
-    Parser& operator =(const Parser&); // do not implement
-
 public:
     typedef SBuf::size_type size_type;
 
-    Parser() { clear(); }
-    virtual ~Parser() {}
+    Parser() : parsingStage_(HTTP_PARSE_NONE) {}
 
     /// Set this parser back to a default state.
     /// Will DROP any reference to a buffer (does not free).

diff --git a/src/http/one/RequestParser.h b/src/http/one/RequestParser.h
index 8f6a51e40d83f26616b2e2a1949f90c8ffb79f09..4481f682103de5c852a0da030a043d6e206b29fb 100644 (file)
--- a/src/http/one/RequestParser.h
+++ b/src/http/one/RequestParser.h
@@ -18,13 +18,9 @@ namespace One {
  */
 class RequestParser : public Http1::Parser
 {
-    explicit RequestParser(const RequestParser&); // do not implement
-    RequestParser& operator =(const RequestParser&); // do not implement
-
 public:
     /* Http::One::Parser API */
     RequestParser() : Parser() {clear();}
-    virtual ~RequestParser() {}
     virtual void clear();
     virtual Http1::Parser::size_type firstLineSize() const {return req.end - req.start + 1;}
     virtual bool parse(const SBuf &aBuf);