Fix annoying build warning

hcore/include/senseApis/senseApiLib.h

@@ -64,12 +64,23 @@ public:
 
 class SenseApiLib
 {
+private:
+    friend class SenseApiManager;
+    struct DlCloser
+    {
+        void operator()(void* handle) const
+        {
+            if (handle) {
+                dlclose(handle);
+            }
+        }
+    };
+
 public:
     SenseApiLib(
         const std::string& path, void *_dlopen_handle, getSenseApiDescFn *descFn)
     :   libraryPath(path),
-        dlopen_handle(
-            _dlopen_handle, reinterpret_cast<void(*)(void*)>(&dlclose)),
+        dlopen_handle(_dlopen_handle, DlCloser()),
         HK_GET_SENSE_API_DESC_FN_NAME(descFn)
     {}
 
@@ -98,7 +109,7 @@ public:
 
 public:
     std::string libraryPath;
-    std::unique_ptr<void, void(*)(void*)> dlopen_handle;
+    std::unique_ptr<void, DlCloser> dlopen_handle;
     /* UNIMPLEMENTED: API-specific cmdline options. These affect this specific
      * sense api lib's behaviour globally.
      */

hcore/include/senseApis/senseApiManager.h

@@ -12,7 +12,8 @@
 namespace hk {
 namespace sense_api {
 
-class SenseApiManager {
+class SenseApiManager
+{
 public:
     static SenseApiManager& getInstance()
     {

hcore/senseApis/senseApiManager.cpp

@@ -12,14 +12,6 @@ namespace fs = std::filesystem;
 namespace hk {
 namespace sense_api {
 
-struct DlCloser {
-    void operator()(void* handle) const {
-        if (handle) {
-            dlclose(handle);
-        }
-    }
-};
-
 static std::optional<std::string> findLibraryPath(
     const std::string& libraryPath)
 {
@@ -64,7 +56,7 @@ SenseApiLib& SenseApiManager::loadSenseApiLib(const std::string& libraryPath)
 
     // Clear any existing error
     dlerror();
-    auto dlopen_handle = std::unique_ptr<void, DlCloser>(
+    auto dlopen_handle = std::unique_ptr<void, SenseApiLib::DlCloser>(
         dlopen(resolvedPath.c_str(), RTLD_LAZY));
     if (!dlopen_handle && fullPath.has_value())
     {