ErasureCodeJerasure.cc: remove redundant checks before erase()

Fix for:

[src/erasure-code/jerasure/ErasureCodeJerasure.cc:254]: (style)
 Redundant checking of STL container element existence before removing it.
[src/erasure-code/jerasure/ErasureCodeJerasure.cc:493]: (style)
 Redundant checking of STL container element existence before removing it.
[src/erasure-code/jerasure/ErasureCodeJerasure.cc:496]: (style)
 Redundant checking of STL container element existence before removing it.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

diff --git a/src/erasure-code/jerasure/ErasureCodeJerasure.cc b/src/erasure-code/jerasure/ErasureCodeJerasure.cc
index eb10254c064cce4dc9f8623e6f89cbfaaf904063..408f51f52e3ed38afe53770096da944ae8247cd7 100644 (file)
--- a/src/erasure-code/jerasure/ErasureCodeJerasure.cc
+++ b/src/erasure-code/jerasure/ErasureCodeJerasure.cc
@@ -251,8 +251,7 @@ int ErasureCodeJerasureReedSolomonRAID6::parse(ErasureCodeProfile &profile,
                                               ostream *ss)
 {
   int err = ErasureCodeJerasure::parse(profile, ss);
-  if (profile.find("m") != profile.end())
-    profile.erase("m");
+  profile.erase("m");
   m = 2;
   if (w != 8 && w != 16 && w != 32) {
     *ss << "ReedSolomonRAID6: w=" << w
@@ -490,11 +489,9 @@ int ErasureCodeJerasureLiber8tion::parse(ErasureCodeProfile &profile,
                                         ostream *ss)
 {
   int err = ErasureCodeJerasure::parse(profile, ss);
-  if (profile.find("m") != profile.end())
-    profile.erase("m");
+  profile.erase("m");
   err |= to_int("m", profile, &m, DEFAULT_M, ss);
-  if (profile.find("w") != profile.end())
-    profile.erase("w");
+  profile.erase("w");
   err |= to_int("w", profile, &w, DEFAULT_W, ss);
   err |= to_int("packetsize", profile, &packetsize, DEFAULT_PACKETSIZE, ss);