Fix cast and add test

Author: Mmanu Chaturvedi

include/pybind11/cast.h

@@ -1970,7 +1970,7 @@ detail::enable_if_t<
 
 template <typename T>
 detail::enable_if_t<
-        std::is_rvalue_reference<T&&>::value && !detail::is_pyobject<detail::intrinsic_t<T>>::value, object>
+        std::is_rvalue_reference<T&&>::value && !std::is_pointer<T>::value && !detail::is_pyobject<detail::intrinsic_t<T>>::value, object>
     cast(T&& value) {
     // Have to use `pybind11::move` because some compilers might try to bind `move` to `std::move`...
     return pybind11::move<T>(std::move(value));

tests/test_builtin_casters.cpp

@@ -167,4 +167,24 @@ TEST_SUBMODULE(builtin_casters, m) {
         py::object o = py::cast(v);
         return py::cast<void *>(o) == v;
     });
+    class A {
+        public:
+        std::string s;
+        int i;
+        std::pair<std::string, int> P;
+    };
+
+    py::class_<A>(m, "A")
+        .def(py::init());
+
+    m.def("test_pointer_caster", []() -> bool {
+        A a;
+        a.s = "String 1";
+        a.i = 100;
+        a.P = std::pair<std::string, int>("String 2", 10);
+        A *a_ptr = &a;
+        py::object o = py::cast(&a);
+        py::object o1 = py::cast(a_ptr);
+        return (py::cast<A*>(o) == a_ptr && py::cast<A*>(o1) == a_ptr);
+    });
 }

tests/test_builtin_casters.py

@@ -340,3 +340,7 @@ def test_int_long():
 
 def test_void_caster_2():
     assert m.test_void_caster()
+
+
+def test_pointer_caster():
+    assert m.test_pointer_caster()