]> git.ipfire.org Git - thirdparty/squid.git/commit
Cleanup helper.h classes (#719)
authorAmos Jeffries <yadij@users.noreply.github.com>
Sat, 17 Oct 2020 05:33:11 +0000 (05:33 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Sat, 17 Oct 2020 07:00:29 +0000 (07:00 +0000)
commit017a2d1f579509bf7ab5ae4ddc6ef2bca48e17c2
tree4e8ec49bbdb22738d9206434f39d076b2d90ec01
parent8ecbe78dd0b13a6a23bfaa066a120ec17342813d
Cleanup helper.h classes (#719)

Polish up the classes in helper.h to use proper constructors as the
caller API for creation + initialization. Use C++11 initialization for
default values.

* no "virtual" in helper class destructor declaration could create
  memory leaks in future (poorly) refactored code; the gained protection
  is probably worth adding the (currently unused) virtual table to the
  helper class; resolves Coverity issue CID 1461141 (VIRTUAL_DTOR)
* missing Comm::Connection timers initialization on helper startup
* multiple initialization of values used for state accounting
* initialization of booleans to non-0 integer values
* initialization of char using numeric values
* missing mgr:filedescriptors description for helper sockets
src/helper.cc
src/helper.h