Improve type safety of internals.registered_types_cpp

Author: dean0x7d

include/pybind11/cast.h

@@ -195,15 +195,15 @@ inline detail::type_info *get_local_type_info(const std::type_index &tp) {
     auto &locals = registered_local_types_cpp();
     auto it = locals.find(tp);
     if (it != locals.end())
-        return (detail::type_info *) it->second;
+        return it->second;
     return nullptr;
 }
 
 inline detail::type_info *get_global_type_info(const std::type_index &tp) {
     auto &types = get_internals().registered_types_cpp;
     auto it = types.find(tp);
     if (it != types.end())
-        return (detail::type_info *) it->second;
+        return it->second;
     return nullptr;
 }
 

include/pybind11/detail/internals.h

@@ -65,7 +65,7 @@ struct overload_hash {
 /// Whenever binary incompatible changes are made to this structure,
 /// `PYBIND11_INTERNALS_VERSION` must be incremented.
 struct internals {
-    type_map<void *> registered_types_cpp; // std::type_index -> type_info
+    type_map<type_info *> registered_types_cpp; // std::type_index -> type_info
     std::unordered_map<PyTypeObject *, std::vector<type_info *>> registered_types_py; // PyTypeObject* -> base type_info(s)
     std::unordered_multimap<const void *, instance*> registered_instances; // void * -> instance*
     std::unordered_set<std::pair<const PyObject *, const char *>, overload_hash> inactive_overload_cache;
@@ -167,8 +167,8 @@ PYBIND11_NOINLINE inline internals &get_internals() {
 }
 
 /// Works like `internals.registered_types_cpp`, but for module-local registered types:
-PYBIND11_NOINLINE inline type_map<void *> &registered_local_types_cpp() {
-    static type_map<void *> locals{};
+PYBIND11_NOINLINE inline type_map<type_info *> &registered_local_types_cpp() {
+    static type_map<type_info *> locals{};
     return locals;
 }