Use default copy constructor and assignment

This fixes one bug in PeerOptions::flags handling

diff --git a/src/security/PeerOptions.cc b/src/security/PeerOptions.cc
index cb0778f31bc6e1e35fbc2ed10383a4c34d027b90..0aa2b5331319ea38118445a9bdb8643d7f1d3939 100644 (file)
--- a/src/security/PeerOptions.cc
+++ b/src/security/PeerOptions.cc
@@ -27,44 +27,6 @@ Security::PeerOptions::PeerOptions()
      parseOptions();
 }
 
-Security::PeerOptions::PeerOptions(const Security::PeerOptions &p) :
-    sslOptions(p.sslOptions),
-    caDir(p.caDir),
-    crlFile(p.crlFile),
-    sslCipher(p.sslCipher),
-    sslFlags(p.sslFlags),
-    sslDomain(p.sslDomain),
-    parsedOptions(p.parsedOptions),
-    parsedFlags(p.parsedFlags),
-    certs(p.certs),
-    caFiles(p.caFiles),
-    parsedCrl(p.parsedCrl),
-    sslVersion(p.sslVersion),
-    encryptTransport(p.encryptTransport)
-{
-    memcpy(&flags, &p.flags, sizeof(flags));
-}
-
-Security::PeerOptions &
-Security::PeerOptions::operator =(const Security::PeerOptions &p)
-{
-    sslOptions = p.sslOptions;
-    caDir = p.caDir;
-    crlFile = p.crlFile;
-    sslCipher = p.sslCipher;
-    sslFlags = p.sslFlags;
-    sslDomain = p.sslDomain;
-    parsedOptions = p.parsedOptions;
-    parsedFlags = p.parsedFlags;
-    certs = p.certs;
-    caFiles = p.caFiles;
-    parsedCrl = p.parsedCrl;
-    sslVersion = p.sslVersion;
-    encryptTransport = p.encryptTransport;
-    memcpy(&flags, &p.flags, sizeof(flags));
-    return *this;
-}
-
 void
 Security::PeerOptions::parse(const char *token)
 {

diff --git a/src/security/PeerOptions.h b/src/security/PeerOptions.h
index c2f86201db376c8ae09a0d66623bd94be95119c6..ca1eabc4fcd187ac8b7f38413799037a9d87627d 100644 (file)
--- a/src/security/PeerOptions.h
+++ b/src/security/PeerOptions.h
@@ -23,8 +23,8 @@ class PeerOptions
 {
 public:
     PeerOptions();
-    PeerOptions(const PeerOptions &);
-    PeerOptions &operator =(const PeerOptions &);
+    PeerOptions(const PeerOptions &) = default;
+    PeerOptions &operator =(const PeerOptions &) = default;
     virtual ~PeerOptions() {}
 
     /// parse a TLS squid.conf option

diff --git a/src/tests/stub_libsecurity.cc b/src/tests/stub_libsecurity.cc
index 28ee94ca8336c4827ebf9c72daba600882fd64dd..0dcff346e7c2be621fb3983c199532dcc50a8b99 100644 (file)
--- a/src/tests/stub_libsecurity.cc
+++ b/src/tests/stub_libsecurity.cc
@@ -74,8 +74,6 @@ Security::PeerOptions::PeerOptions() {
 #endif
      STUB_NOP
 }
-Security::PeerOptions::PeerOptions(const Security::PeerOptions &) {STUB}
-Security::PeerOptions &Security::PeerOptions::operator =(const Security::PeerOptions &) STUB_RETVAL(*this)
 void Security::PeerOptions::parse(char const*) STUB
 Security::ContextPointer Security::PeerOptions::createClientContext(bool) STUB_RETVAL(Security::ContextPointer())
 void Security::PeerOptions::updateTlsVersionLimits() STUB