gh-509: Remove deprecated utils, celixbool, export and celix_utils_api header

    Also refactor code using the removed depecrated headers

Author: pnoltes

CHANGES.md

@@ -58,6 +58,9 @@ limitations under the License.
 - linked_list.h is removed and no longer supported. Use celix_array_list.h instead.
 - ip_utils.h is removed and no longer supported.  
 - array_list.h is removed and no longer supported. Use celix_array_list.h instead.
+- utils.h is removed and no longer supported. Use celix_utils.h instead.
+- export.h is removed and no longer supported.
+- celix_utils_api.h is removed and no longer supported.
 - the celix_arrayList_add function no longer accepts a NULL value.
 - version.h and version_range.h are removed and no longer supported. Use celix_version.h and celix_version_range.h 
   instead.

bundles/remote_services/discovery_common/src/endpoint_discovery_poller.c

@@ -35,7 +35,6 @@
 #include "bundle_context.h"
 #include "celix_log_helper.h"
 #include "celix_utils.h"
-#include "utils.h"
 
 #include "endpoint_descriptor_reader.h"
 #include "discovery.h"
@@ -95,7 +94,11 @@ celix_status_t endpointDiscoveryPoller_create(discovery_t *discovery, celix_bund
 	(*poller)->poll_timeout = atoi(timeout);
 	(*poller)->discovery = discovery;
 	(*poller)->running = false;
-	(*poller)->entries = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+	(*poller)->entries = hashMap_create(
+        (unsigned int (*)(const void*))celix_utils_stringHash,
+        NULL,
+        (int (*)(const void*, const void*))celix_utils_stringEquals,
+        NULL);
 
 	const char* sep = ",";
 	char *save_ptr = NULL;

bundles/remote_services/discovery_common/src/endpoint_discovery_server.c

@@ -27,7 +27,6 @@
 #include "civetweb.h"
 #include "celix_errno.h"
 #include "celix_utils.h"
-#include "utils.h"
 #include "celix_log_helper.h"
 #include "discovery.h"
 #include "endpoint_descriptor_writer.h"
@@ -87,7 +86,11 @@ celix_status_t endpointDiscoveryServer_create(discovery_t *discovery,
     }
 
     (*server)->loghelper = &discovery->loghelper;
-    (*server)->entries = hashMap_create(&utils_stringHash, NULL, &utils_stringEquals, NULL);
+    (*server)->entries = hashMap_create(
+        (unsigned int (*)(const void*))celix_utils_stringHash,
+        NULL,
+        (int (*)(const void*, const void*))celix_utils_stringEquals,
+        NULL);
     if (!(*server)->entries) {
         return CELIX_ENOMEM;
     }

bundles/remote_services/discovery_configured/src/discovery_impl.c

@@ -31,7 +31,7 @@
 
 #include "celix_threads.h"
 #include "bundle_context.h"
-#include "utils.h"
+#include "celix_utils.h"
 #include "celix_log_helper.h"
 
 #include "discovery.h"
@@ -52,7 +52,11 @@ celix_status_t discovery_create(celix_bundle_context_t *context, discovery_t **d
 		(*discovery)->stopped = false;
 
 		(*discovery)->listenerReferences = hashMap_create(serviceReference_hashCode, NULL, serviceReference_equals2, NULL);
-		(*discovery)->discoveredServices = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+		(*discovery)->discoveredServices = hashMap_create(
+            (unsigned int (*)(const void*))celix_utils_stringHash,
+            NULL,
+            (int (*)(const void*, const void*))celix_utils_stringEquals,
+            NULL);
 
 		celixThreadMutex_create(&(*discovery)->mutex, NULL);
 

bundles/remote_services/discovery_etcd/src/discovery_impl.c

@@ -27,7 +27,7 @@
 
 #include "celix_threads.h"
 #include "bundle_context.h"
-#include "utils.h"
+#include "celix_utils.h"
 #include "celix_errno.h"
 #include "filter.h"
 #include "service_reference.h"
@@ -56,7 +56,11 @@ celix_status_t discovery_create(celix_bundle_context_t *context, discovery_t** o
 
         discovery->listenerReferences = hashMap_create(serviceReference_hashCode, NULL, serviceReference_equals2,
                                                           NULL);
-        discovery->discoveredServices = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+        discovery->discoveredServices = hashMap_create(
+            (unsigned int (*)(const void*))celix_utils_stringHash,
+            NULL,
+            (int (*)(const void*, const void*))celix_utils_stringEquals,
+            NULL);
 
         celixThreadMutex_create(&discovery->mutex, NULL);
 
@@ -175,4 +179,3 @@ celix_status_t discovery_stop(discovery_t *discovery) {
 }
 
 
-

bundles/remote_services/discovery_etcd/src/etcd_watcher.c

@@ -24,7 +24,7 @@
 
 #include "celix_api.h"
 #include "celix_log_helper.h"
-#include "utils.h"
+#include "celix_utils.h"
 #include "discovery.h"
 #include "discovery_impl.h"
 
@@ -293,7 +293,11 @@ celix_status_t etcdWatcher_create(discovery_t *discovery, celix_bundle_context_t
 	{
 		(*watcher)->discovery = discovery;
 		(*watcher)->loghelper = &discovery->loghelper;
-		(*watcher)->entries = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+		(*watcher)->entries = hashMap_create(
+            (unsigned int (*)(const void*))celix_utils_stringHash,
+            NULL,
+            (int (*)(const void*, const void*))celix_utils_stringEquals,
+            NULL);
         (*watcher)->ttl = DEFAULT_ETCD_TTL;
 	}
 
@@ -352,4 +356,3 @@ celix_status_t etcdWatcher_destroy(etcd_watcher_t *watcher) {
 
 	return status;
 }
-

bundles/remote_services/remote_service_admin_dfi/src/remote_service_admin_dfi.c

@@ -32,8 +32,8 @@
 
 #include <jansson.h>
 #include "json_serializer.h"
-#include "utils.h"
 #include "celix_utils.h"
+#include "celix_threads.h"
 #include "celix_ref.h"
 #include "hash_map.h"
 
@@ -1159,4 +1159,3 @@ static size_t remoteServiceAdmin_write(void *contents, size_t size, size_t nmemb
     fwrite(contents, size, nmemb, get->stream);
     return size * nmemb;
 }
-

bundles/remote_services/topology_manager/src/scope.c

@@ -23,7 +23,7 @@
 #include "scope.h"
 #include "tm_scope.h"
 #include "topology_manager.h"
-#include "utils.h"
+#include "celix_utils.h"
 #include "filter.h"
 
 static bool import_equal(celix_array_list_entry_t src, celix_array_list_entry_t dest);
@@ -198,7 +198,11 @@ celix_status_t scope_scopeCreate(void *handle, scope_pt *scope) {
     celixThreadMutex_create(&(*scope)->exportScopeLock, NULL);
     celixThreadMutex_create(&(*scope)->importScopeLock, NULL);
 
-    (*scope)->exportScopes = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+    (*scope)->exportScopes = hashMap_create(
+        (unsigned int (*)(const void*))celix_utils_stringHash,
+        NULL,
+        (int (*)(const void*, const void*))celix_utils_stringEquals,
+        NULL);
     celix_array_list_create_options_t opts = CELIX_EMPTY_ARRAY_LIST_CREATE_OPTIONS;
     opts.equalsCallback = import_equal;
     (*scope)->importScopes = celix_arrayList_createWithOptions(&opts);

bundles/remote_services/topology_manager/src/topology_manager.c

@@ -43,7 +43,7 @@
 #include "remote_constants.h"
 #include "filter.h"
 #include "listener_hook_service.h"
-#include "utils.h"
+#include "celix_utils.h"
 #include "service_reference.h"
 #include "service_registration.h"
 #include "celix_log_helper.h"

bundles/shell/shell/src/c_shell.c

@@ -26,7 +26,6 @@
 #include "celix_utils.h"
 #include "celix_errno.h"
 #include "shell_private.h"
-#include "utils.h"
 
 shell_t* shell_create(celix_bundle_context_t *ctx) {
     shell_t *shell = calloc(1, sizeof(*shell));
@@ -35,7 +34,11 @@ shell_t* shell_create(celix_bundle_context_t *ctx) {
     shell->logHelper = celix_logHelper_create(ctx, "celix_shell");
 
     celixThreadRwlock_create(&shell->lock, NULL);
-    shell->commandServices = hashMap_create(utils_stringHash, NULL, utils_stringEquals, NULL);
+    shell->commandServices = hashMap_create(
+        (unsigned int (*)(const void*))celix_utils_stringHash,
+        NULL,
+        (int (*)(const void*, const void*))celix_utils_stringEquals,
+        NULL);
 
     return shell;
 }