diff options
author | Dong-hee Na <donghee.na@python.org> | 2022-05-21 14:52:45 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-05-21 14:52:45 (GMT) |
commit | dd923c57252ee90306a6fe7401ffc5a9cdc91485 (patch) | |
tree | c8fee21e08f8951a4947d359cb9cef6fa3626b1b | |
parent | d9a48d2b413194f36111aa54df331d60c5bfde3f (diff) | |
download | cpython-dd923c57252ee90306a6fe7401ffc5a9cdc91485.zip cpython-dd923c57252ee90306a6fe7401ffc5a9cdc91485.tar.gz cpython-dd923c57252ee90306a6fe7401ffc5a9cdc91485.tar.bz2 |
[3.11] GH-92898: Make _Py_Cast C++ version compatible with cast operator (gh-92951) (gh-93049)
-rw-r--r-- | Include/pyport.h | 28 | ||||
-rw-r--r-- | Lib/test/_testcppext.cpp | 9 |
2 files changed, 35 insertions, 2 deletions
diff --git a/Include/pyport.h b/Include/pyport.h index 614a278..086ed42 100644 --- a/Include/pyport.h +++ b/Include/pyport.h @@ -26,8 +26,32 @@ // _Py_CAST(const PyObject*, expr) fails with a compiler error. #ifdef __cplusplus # define _Py_STATIC_CAST(type, expr) static_cast<type>(expr) -# define _Py_CAST(type, expr) \ - const_cast<type>(reinterpret_cast<const type>(expr)) + +extern "C++" { +namespace { +template <typename type, typename expr_type> +inline type _Py_reinterpret_cast_impl(expr_type *expr) { + return reinterpret_cast<type>(expr); +} + +template <typename type, typename expr_type> +inline type _Py_reinterpret_cast_impl(expr_type const *expr) { + return reinterpret_cast<type>(const_cast<expr_type *>(expr)); +} + +template <typename type, typename expr_type> +inline type _Py_reinterpret_cast_impl(expr_type &expr) { + return static_cast<type>(expr); +} + +template <typename type, typename expr_type> +inline type _Py_reinterpret_cast_impl(expr_type const &expr) { + return static_cast<type>(const_cast<expr_type &>(expr)); +} +} // namespace +} +# define _Py_CAST(type, expr) _Py_reinterpret_cast_impl<type>(expr) + #else # define _Py_STATIC_CAST(type, expr) ((type)(expr)) # define _Py_CAST(type, expr) ((type)(expr)) diff --git a/Lib/test/_testcppext.cpp b/Lib/test/_testcppext.cpp index f38b487..f6049ee 100644 --- a/Lib/test/_testcppext.cpp +++ b/Lib/test/_testcppext.cpp @@ -40,6 +40,15 @@ test_api_casts(PyObject *Py_UNUSED(module), PyObject *Py_UNUSED(args)) PyTypeObject *type = Py_TYPE(const_obj); assert(Py_REFCNT(const_obj) >= 1); + struct PyObjectProxy { + PyObject* obj; + operator PyObject *() { return obj; } + } proxy_obj = { obj }; + Py_INCREF(proxy_obj); + Py_DECREF(proxy_obj); + assert(Py_REFCNT(proxy_obj) >= 1); + + assert(type == &PyTuple_Type); assert(PyTuple_GET_SIZE(const_obj) == 2); PyObject *one = PyTuple_GET_ITEM(const_obj, 0); |