test/test_rgw_admin*: don't compare unsigend to less than 0

Make checks simpler, use empty() instead of 'length() <= 0'.

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

diff --git a/src/test/test_rgw_admin_meta.cc b/src/test/test_rgw_admin_meta.cc
index 9695a3c6d0c92a20d33e695421d22e79320cbcf9..c0ade77ab48f6467a286a16698ac14f93e0d630b 100644 (file)
--- a/src/test/test_rgw_admin_meta.cc
+++ b/src/test/test_rgw_admin_meta.cc
@@ -132,8 +132,7 @@ int test_helper::extract_input(int argc, char *argv[]){
       ERR_CHECK_NEXT_PARAM(rgw_admin_path);
     }else return -1;
   }
-  if(host.length() <= 0 ||
-     rgw_admin_path.length() <= 0)
+  if(host.empty() || rgw_admin_path.empty())
     return -1;
   return 0;
 }

diff --git a/src/test/test_rgw_admin_opstate.cc b/src/test/test_rgw_admin_opstate.cc
index 464bd85a86ab2670a093ead68e3163191f86227a..b999bb68982df00f0f640b217be95a15ffef82a5 100644 (file)
--- a/src/test/test_rgw_admin_opstate.cc
+++ b/src/test/test_rgw_admin_opstate.cc
@@ -135,8 +135,7 @@ int test_helper::extract_input(int argc, char *argv[]){
       ERR_CHECK_NEXT_PARAM(rgw_admin_path);
     }else return -1;
   }
-  if(host.length() <= 0 ||
-     rgw_admin_path.length() <= 0)
+  if(host.empty() || rgw_admin_path.empty())
     return -1;
   return 0;
 }