Make py::repr() a free function

Author: dean0x7d

include/pybind11/pytypes.h

@@ -70,7 +70,6 @@ class object_api : public pyobject_tag {
     bool is_none() const { return derived().ptr() == Py_None; }
     PYBIND11_DEPRECATED("Instead of obj.str(), use py::str(obj)")
     pybind11::str str() const;
-    pybind11::str repr() const;
 
     int ref_count() const { return static_cast<int>(Py_REFCNT(derived().ptr())); }
     handle get_type() const;
@@ -833,6 +832,17 @@ inline size_t len(handle h) {
     return (size_t) result;
 }
 
+inline str repr(handle h) {
+    PyObject *str_value = PyObject_Repr(h.ptr());
+    if (!str_value) throw error_already_set();
+#if PY_MAJOR_VERSION < 3
+    PyObject *unicode = PyUnicode_FromEncodedObject(str_value, "utf-8", nullptr);
+    Py_XDECREF(str_value); str_value = unicode;
+    if (!str_value) throw error_already_set();
+#endif
+    return {str_value, false};
+}
+
 NAMESPACE_BEGIN(detail)
 template <typename D> iterator object_api<D>::begin() const { return {PyObject_GetIter(derived().ptr()), false}; }
 template <typename D> iterator object_api<D>::end() const { return {nullptr, false}; }
@@ -848,16 +858,6 @@ template <typename D> template <typename T> bool object_api<D>::contains(T &&key
 template <typename D>
 pybind11::str object_api<D>::str() const { return pybind11::str(derived()); }
 
-template <typename D>
-pybind11::str object_api<D>::repr() const {
-    PyObject *str_value = PyObject_Repr(derived().ptr());
-#if PY_MAJOR_VERSION < 3
-    PyObject *unicode = PyUnicode_FromEncodedObject(str_value, "utf-8", nullptr);
-    Py_XDECREF(str_value); str_value = unicode;
-#endif
-    return {str_value, false};
-}
-
 template <typename D>
 handle object_api<D>::get_type() const { return (PyObject *) Py_TYPE(derived().ptr()); }
 

tests/test_python_types.cpp

@@ -154,7 +154,7 @@ class ExamplePythonTypes {
 
     void test_print(const py::object& obj) {
         py::print(py::str(obj));
-        py::print(obj.repr());
+        py::print(py::repr(obj));
     }
 
     static int value;