]> git.ipfire.org Git - thirdparty/squid.git/commitdiff
Convert packing methods from Packer* to Packable* objects
authorAmos Jeffries <squid3@treenet.co.nz>
Sun, 26 Apr 2015 16:41:06 +0000 (09:41 -0700)
committerAmos Jeffries <squid3@treenet.co.nz>
Sun, 26 Apr 2015 16:41:06 +0000 (09:41 -0700)
Packaging code can now receive any object that implements the Packable
interface instead of requiring the Packer trampoline object. For now it
is still the only object implementing Packable.

1  2 
src/HttpMsg.cc
src/HttpMsg.h
src/HttpReply.cc
src/HttpReply.h
src/HttpRequest.cc
src/HttpRequest.h
src/tests/stub_HttpReply.cc
src/tests/stub_HttpRequest.cc

diff --cc src/HttpMsg.cc
Simple merge
diff --cc src/HttpMsg.h
index d85aa795b8445ea97c60d9824dd4188485a8d0f4,275b260591671e9b3188d2143ad0b144451720f0..065820dddb6de19b00759d9278be9cf8de64d54c
@@@ -89,9 -89,9 +89,9 @@@ protected
       * \retval true   Status line has no serious problems.
       * \retval false  Status line has a serious problem. Correct response is indicated by error.
       */
 -    virtual bool sanityCheckStartLine(MemBuf *buf, const size_t hdr_len, Http::StatusCode *error) = 0;
 +    virtual bool sanityCheckStartLine(const char *buf, const size_t hdr_len, Http::StatusCode *error) = 0;
  
-     virtual void packFirstLineInto(Packer * p, bool full_uri) const = 0;
+     virtual void packFirstLineInto(Packable * p, bool full_uri) const = 0;
  
      virtual bool parseFirstLine(const char *blk_start, const char *blk_end) = 0;
  
Simple merge
diff --cc src/HttpReply.h
Simple merge
Simple merge
index 2c104b1bb02ccae0d930dc5d916cca705b9c75c0,ada6ed0560283e0041a5c51bb1d46eeab45ff676..526a16ef97e72fb3a72e4163df282b50aa2b7f9a
@@@ -255,9 -255,9 +255,9 @@@ private
      mutable int64_t rangeOffsetLimit;  /* caches the result of getRangeOffsetLimit */
  
  protected:
-     virtual void packFirstLineInto(Packer * p, bool full_uri) const;
+     virtual void packFirstLineInto(Packable * p, bool full_uri) const;
  
 -    virtual bool sanityCheckStartLine(MemBuf *buf, const size_t hdr_len, Http::StatusCode *error);
 +    virtual bool sanityCheckStartLine(const char *buf, const size_t hdr_len, Http::StatusCode *error);
  
      virtual void hdrCacheInit();
  
index 8ae8d38863e5838d7efa09af6b38468724c95932,a91e28027a19d46d6b7e87b2595d5313769cb34d..b8dabc9135ac1230f9a3e7334330516ec4787bba
@@@ -18,10 -18,10 +18,10 @@@ HttpReply::HttpReply() : HttpMsg(hoRepl
      STUB_NOP
      HttpReply::~HttpReply() STUB
      void HttpReply::setHeaders(Http::StatusCode status, const char *reason, const char *ctype, int64_t clen, time_t lmt, time_t expires_) STUB
-     void HttpReply::packHeadersInto(Packer * p) const STUB
+     void HttpReply::packHeadersInto(Packable *) const STUB
      void HttpReply::reset() STUB
-     void httpBodyPackInto(const HttpBody * body, Packer * p) STUB
+     void httpBodyPackInto(const HttpBody *, Packable *) STUB
 -    bool HttpReply::sanityCheckStartLine(MemBuf *buf, const size_t hdr_len, Http::StatusCode *error) STUB_RETVAL(false)
 +    bool HttpReply::sanityCheckStartLine(const char *buf, const size_t hdr_len, Http::StatusCode *error) STUB_RETVAL(false)
      int HttpReply::httpMsgParseError() STUB_RETVAL(0)
      bool HttpReply::expectingBody(const HttpRequestMethod&, int64_t&) const STUB_RETVAL(false)
      bool HttpReply::parseFirstLine(const char *start, const char *end) STUB_RETVAL(false)
index 62ff173da5105e4860561fad6f0bfa85d1d13920,9c45144062c3eaf6ce21e74995832f53787624aa..765769aaddb7e5625880940879c831c5cf4d0a22
@@@ -16,8 -16,8 +16,8 @@@
  HttpRequest::HttpRequest() : HttpMsg(hoRequest) STUB
      HttpRequest::HttpRequest(const HttpRequestMethod& aMethod, AnyP::ProtocolType aProtocol, const char *aUrlpath) : HttpMsg(hoRequest) STUB
      HttpRequest::~HttpRequest() STUB
-     void HttpRequest::packFirstLineInto(Packer * p, bool full_uri) const STUB
+     void HttpRequest::packFirstLineInto(Packable *, bool) const STUB
 -    bool HttpRequest::sanityCheckStartLine(MemBuf *buf, const size_t hdr_len, Http::StatusCode *error) STUB_RETVAL(false)
 +    bool HttpRequest::sanityCheckStartLine(const char*buf, const size_t hdr_len, Http::StatusCode *error) STUB_RETVAL(false)
      void HttpRequest::hdrCacheInit() STUB
      void HttpRequest::reset() STUB
      bool HttpRequest::expectingBody(const HttpRequestMethod& unused, int64_t&) const STUB_RETVAL(false)