move argument checking to cconvert

Co-authored-by: Valentin Churavy <[email protected]>
cesmix-mit · Aug 7, 2024 · 5874369 · 5874369
1 parent 59d208b
commit 5874369
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/LAMMPS.jl b/src/LAMMPS.jl
@@ -146,11 +146,12 @@ mutable struct LMP
     end
 end
 
-function Base.unsafe_convert(::Type{Ptr{Cvoid}}, lmp::LMP)    
+function Base.cconvert(::Type{Ptr{Cvoid}}, lmp::LMP)    
     lmp.handle == C_NULL && error("The LMP object doesn't point to a valid LAMMPS instance! "
             * "This is usually caused by calling `LAMMPS.close!` or through serialization and deserialization.")
-    return lmp.handle
+    return lmp
 end
+Base.unsafe_convert(::Type{Ptr{Cvoid}}, lmp::LMP) = lmp.handle
 
 """
     close!(lmp::LMP)