Polished HelperChildConfig methods. No runtime changes expected.

Removed 3 out of 5 constructors as unused.

Prevented silent conversion from int to HelperChildConfig via a
single-parameter constructor.

Removed destructor and assignment operator for being identical to the
compiler-generated ones and for implying that a copy constructor is missing.

diff --git a/src/HelperChildConfig.cc b/src/HelperChildConfig.cc
index 85fdf0ebd7f4e6d7a96801cfa350f452d94cb62e..71c00442fe4b1d5edd66392e1d48b1daeb36f522 100644 (file)
--- a/src/HelperChildConfig.cc
+++ b/src/HelperChildConfig.cc
@@ -4,31 +4,15 @@
 
 #include <string.h>
 
-HelperChildConfig::HelperChildConfig(const unsigned int m, const unsigned int s, const unsigned int i, const unsigned int cc) :
+HelperChildConfig::HelperChildConfig(const unsigned int m):
         n_max(m),
-        n_startup(s),
-        n_idle(i),
-        concurrency(cc),
+        n_startup(0),
+        n_idle(1),
+        concurrency(0),
         n_running(0),
         n_active(0)
 {}
 
-HelperChildConfig::~HelperChildConfig()
-{}
-
-HelperChildConfig &
-HelperChildConfig::operator =(const HelperChildConfig &rhs)
-{
-    // Copies everything.
-    n_max = rhs.n_max;
-    n_startup = rhs.n_startup;
-    n_idle = rhs.n_idle;
-    concurrency = rhs.concurrency;
-    n_running = rhs.n_running;
-    n_active = rhs.n_active;
-    return *this;
-}
-
 HelperChildConfig &
 HelperChildConfig::updateLimits(const HelperChildConfig &rhs)
 {

diff --git a/src/HelperChildConfig.h b/src/HelperChildConfig.h
index a6ee3980295b7bedcc210887a4d1bad3ecd78280..d8f4045202ef7f7e67fea9b7bba7e7d4a7f52f97 100644 (file)
--- a/src/HelperChildConfig.h
+++ b/src/HelperChildConfig.h
@@ -10,9 +10,7 @@
 class HelperChildConfig
 {
 public:
-    HelperChildConfig(const unsigned int m=0, const unsigned int s=0, const unsigned int i=1, const unsigned int cc=0);
-    ~HelperChildConfig();
-    HelperChildConfig &operator =(const HelperChildConfig &rhs);
+    explicit HelperChildConfig(const unsigned int m = 0);
 
     /**
      * When new helpers are needed call this to find out how many more
@@ -62,7 +60,6 @@ public:
     unsigned int concurrency;
 
     /* derived from active operations */
-public:
 
     /**
      * Total helper children objects currently existing.

diff --git a/src/tests/stub_HelperChildConfig.cc b/src/tests/stub_HelperChildConfig.cc
index 6c2eb1bc0346c12133c2f9cdffe6cebf1f75c07c..23538ce384dc9a0b39d3db84e9c3999a69b0030f 100644 (file)
--- a/src/tests/stub_HelperChildConfig.cc
+++ b/src/tests/stub_HelperChildConfig.cc
@@ -4,25 +4,15 @@
 
 #include <string.h>
 
-HelperChildConfig::HelperChildConfig(const unsigned int m, const unsigned int s, const unsigned int i, const unsigned int cc) :
+HelperChildConfig::HelperChildConfig(const unsigned int m):
         n_max(m),
-        n_startup(s),
-        n_idle(i),
-        concurrency(cc),
+        n_startup(0),
+        n_idle(1),
+        concurrency(0),
         n_running(0),
         n_active(0)
 {}
 
-HelperChildConfig::~HelperChildConfig()
-{}
-
-HelperChildConfig &
-HelperChildConfig::operator =(const HelperChildConfig &rhs)
-{
-    memcpy(this, &rhs, sizeof(HelperChildConfig));
-    return *this;
-}
-
 int
 HelperChildConfig::needNew() const
 {