Drop unnecessary code

Author: KFilipek

src/memory_pool.c

@@ -54,7 +54,7 @@ static int CTL_SUBTREE_HANDLER(default)(void *ctx,
                                         umf_ctl_index_utlist_t *indexes,
                                         const char *extra_name,
                                         umf_ctl_query_type_t queryType) {
-    (void)indexes, (void)source, (void)ctx;
+    (void)indexes, (void)source;
     utils_init_once(&mem_pool_ctl_initialized, ctl_init);
     utils_mutex_lock(&ctl_mtx);
 
@@ -93,6 +93,7 @@ static int CTL_SUBTREE_HANDLER(default)(void *ctx,
             return UMF_RESULT_ERROR_INVALID_ARGUMENT;
         }
     }
+
     utils_mutex_unlock(&ctl_mtx);
     return 0;
 }

src/pool/pool_disjoint.c

@@ -36,7 +36,7 @@ static int CTL_READ_HANDLER(name)(void *ctx, umf_ctl_query_source_t source,
                                   umf_ctl_index_utlist_t *indexes,
                                   const char *extra_name,
                                   umf_ctl_query_type_t queryType) {
-    (void)source, (void)indexes, (void)queryType, (void)size, (void)extra_name;
+    (void)source, (void)indexes, (void)queryType, (void)extra_name;
     disjoint_pool_t *pool = (disjoint_pool_t *)ctx;
 
     if (arg == NULL) {

test/ctl/ctl_api.cpp

@@ -196,18 +196,6 @@ class CtlTest : public ::testing::Test {
   private:
 };
 
-TEST_F(CtlTest, ctlDefaultNegative) {
-    void *ctx = (void *)0xBABE;
-    void *arg = (void *)0xBABE;
-
-    // Test invalid path for umfCtlGet and umfCtlSet
-    auto res = umfCtlGet("umf.pool.default.disjoint.some_path", ctx, arg, 0);
-    ASSERT_EQ(res, UMF_RESULT_ERROR_INVALID_ARGUMENT);
-
-    res = umfCtlSet("umf.pool.default.disjoint.some_path", ctx, arg, 0);
-    ASSERT_EQ(res, UMF_RESULT_ERROR_INVALID_ARGUMENT);
-}
-
 /* Case: default settings
  * This test sets a default value and then retrieves it */
 TEST_F(CtlTest, ctlDefault) {