Refactor

crates/hotload_plugin/src/lib.rs

@@ -1,6 +1,5 @@
-#[repr(C)]
 #[derive(Debug, Clone)]
-pub struct HotloadPlugin {
+pub struct HotloadPluginInfo {
     // The plugin's name
     pub name: String,
 

plugins/hotload_toto1/src/lib.rs

@@ -1,8 +1,8 @@
-use hotload_plugin::HotloadPlugin;
+use hotload_plugin::HotloadPluginInfo;
 
 #[no_mangle]
-pub extern "C" fn register() -> *const HotloadPlugin {
+pub extern fn plugin_info() -> *const HotloadPluginInfo {
-    let plugin = Box::new(HotloadPlugin {
+    let plugin = Box::new(HotloadPluginInfo {
         name: String::from("test_lib"),
         version: String::from("0.1.0"),
         author: String::from("mrdev023"),

src/main.rs

@@ -3,10 +3,8 @@ mod plugin_manager;
 fn main() {
     let plugin_manager = plugin_manager::PluginManager::new();
 
-    unsafe {
+    for plugin in &plugin_manager.plugins {
-        for plugin in &plugin_manager.plugins {
+        println!("Plugin {:?}", plugin.get_info().unwrap());
-            println!("Plugin {:?}", (*(*plugin)));
-        }
     }
 
     println!("Plugins number : {}", plugin_manager.plugins.len());

src/plugin_manager.rs

@@ -1,10 +1,35 @@
 use glob::glob;
-use hotload_plugin::HotloadPlugin;
+use hotload_plugin::HotloadPluginInfo;
 use std::env::current_exe;
 use std::path::PathBuf;
 
+pub struct Plugin {
+    library: libloading::Library,
+}
+
+impl Plugin {
+    pub fn new(library: libloading::Library) -> Self {
+        Self { library }
+    }
+
+    pub fn get_info(&self) -> Result<HotloadPluginInfo, String> {
+        let get_info_fn: libloading::Symbol<extern "C" fn() -> *const HotloadPluginInfo> = unsafe {
+            self.library
+                .get(b"plugin_info\0")
+                .map_err(|e| format!("{}", e))?
+        };
+
+        let info = get_info_fn();
+
+        if info.is_null() {
+            return Err(String::from("get_info function returned null"));
+        }
+        Ok(unsafe { (*info).clone() })
+    }
+}
+
 pub struct PluginManager {
-    pub plugins: Vec<*const HotloadPlugin>,
+    pub plugins: Vec<Plugin>,
 }
 
 impl PluginManager {
@@ -22,13 +47,11 @@ impl PluginManager {
         PluginManager { plugins }
     }
 
-    fn load_library(lib_file: PathBuf) -> *const HotloadPlugin {
+    fn load_library(lib_file: PathBuf) -> Plugin {
-        println!("Loading library {:?}", lib_file);
+        println!("Loading library {:?}", lib_file.file_name().unwrap());
         unsafe {
             let lib = libloading::Library::new(lib_file).unwrap();
-            let register_func: libloading::Symbol<extern "C" fn() -> *const HotloadPlugin> =
+            Plugin::new(lib)
-                lib.get(b"register").unwrap();
-            register_func()
         }
     }
 }