Fix Value::String::to_pointer for Lua < 5.4

mlua-rs · Oct 18, 2024 · 930fd9c · 930fd9c
1 parent 2a8db87
commit 930fd9c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/value.rs b/src/value.rs
@@ -129,9 +129,14 @@ impl Value {
     #[inline]
     pub fn to_pointer(&self) -> *const c_void {
         match self {
+            Value::String(String(vref)) => {
+                // In Lua < 5.4 (excluding Luau), string pointers are NULL
+                // Use alternative approach
+                let lua = vref.lua.lock();
+                unsafe { ffi::lua_tostring(lua.ref_thread(), vref.index) as *const c_void }
+            }
             Value::LightUserData(ud) => ud.0,
-            Value::String(String(vref))
-            | Value::Table(Table(vref))
+            Value::Table(Table(vref))
             | Value::Function(Function(vref))
             | Value::Thread(Thread(vref, ..))
             | Value::UserData(AnyUserData(vref))