Use copy instead of deepcopy for copy constructor

Author: barche

include/jlcxx/jlcxx_config.hpp

@@ -14,8 +14,8 @@
 #endif
 
 #define JLCXX_VERSION_MAJOR 0
-#define JLCXX_VERSION_MINOR 7
-#define JLCXX_VERSION_PATCH 2
+#define JLCXX_VERSION_MINOR 8
+#define JLCXX_VERSION_PATCH 0
 
 // From https://stackoverflow.com/questions/5459868/concatenate-int-to-string-using-c-preprocessor
 #define __JLCXX_STR_HELPER(x) #x

include/jlcxx/module.hpp

@@ -669,7 +669,7 @@ class JLCXX_API Module
     if constexpr (CopyConstructible<T>::value)
     {
       set_override_module(jl_base_module);
-      method("deepcopy_internal", [this](const T& other, ObjectIdDict)
+      method("copy", [this](const T& other)
       {
         return create<T>(other);
       });

include/jlcxx/type_conversion.hpp

@@ -999,24 +999,6 @@ struct julia_type_factory<BoxedValue<T>>
   static jl_datatype_t* julia_type() { return jl_any_type; }
 };
 
-// Helper for ObjectIdDict
-struct ObjectIdDict {};
-
-template<> struct static_type_mapping<ObjectIdDict>
-{
-  typedef jl_value_t* type;
-};
-
-// Used for deepcopy_internal overloading
-template<>
-struct ConvertToCpp<ObjectIdDict>
-{
-  ObjectIdDict operator()(jl_value_t*) const
-  {
-    return ObjectIdDict();
-  }
-};
-
 /// Helper for Singleton types (Type{T} in Julia)
 template<typename T>
 struct SingletonType

src/jlcxx.cpp

@@ -374,7 +374,6 @@ JLCXX_API void register_core_types()
 
     jlcxx::detail::AddIntegerTypes<fixed_int_types>()("Int", "");
 
-    set_julia_type<ObjectIdDict>((jl_datatype_t*)julia_type("IdDict", jl_base_module),false);
     set_julia_type<jl_datatype_t*>(jl_any_type,false);
     set_julia_type<jl_value_t*>(jl_any_type,false);
     registered = true;