diff options
author | Cristian Maureira-Fredes <cristian.maureira-fredes@qt.io> | 2018-11-13 12:15:06 +0100 |
---|---|---|
committer | Cristian Maureira-Fredes <cristian.maureira-fredes@qt.io> | 2018-11-19 12:44:30 +0000 |
commit | 3c2ef04c9eade4226f139064c6c45cb4819834e0 (patch) | |
tree | e46d163147df58b8c338d4ee992287978739f7e6 /sources/shiboken2/libshiboken | |
parent | 9fad5661c40497305cdec5a4aa51a8726675cf90 (diff) |
Add std::nullptr_t support
Shiboken did not have support for `nullptr_t` so the
converter was failing when encountering a `nullptr`.
A test case is included.
Fixes: PYSIDE-854
Change-Id: If5aad245e7074ed791bfc7a42a4c6a56de441d5b
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
Diffstat (limited to 'sources/shiboken2/libshiboken')
-rw-r--r-- | sources/shiboken2/libshiboken/sbkconverter.cpp | 4 | ||||
-rw-r--r-- | sources/shiboken2/libshiboken/sbkconverter.h | 3 | ||||
-rw-r--r-- | sources/shiboken2/libshiboken/sbkconverter_p.h | 30 |
3 files changed, 36 insertions, 1 deletions
diff --git a/sources/shiboken2/libshiboken/sbkconverter.cpp b/sources/shiboken2/libshiboken/sbkconverter.cpp index 65844151f..4f6ebf65f 100644 --- a/sources/shiboken2/libshiboken/sbkconverter.cpp +++ b/sources/shiboken2/libshiboken/sbkconverter.cpp @@ -78,7 +78,8 @@ void init() Primitive<unsigned int>::createConverter(), Primitive<unsigned long>::createConverter(), Primitive<unsigned short>::createConverter(), - VoidPtr::createConverter() + VoidPtr::createConverter(), + Primitive<std::nullptr_t>::createConverter() }; PrimitiveTypeConverters = primitiveTypeConverters; @@ -100,6 +101,7 @@ void init() converters["unsigned long"] = primitiveTypeConverters[SBK_UNSIGNEDLONG_IDX]; converters["unsigned short"] = primitiveTypeConverters[SBK_UNSIGNEDSHORT_IDX]; converters["void*"] = primitiveTypeConverters[SBK_VOIDPTR_IDX]; + converters["std::nullptr_t"] = primitiveTypeConverters[SBK_NULLPTR_T_IDX]; initArrayConverters(); } diff --git a/sources/shiboken2/libshiboken/sbkconverter.h b/sources/shiboken2/libshiboken/sbkconverter.h index 0effebf57..33c33025b 100644 --- a/sources/shiboken2/libshiboken/sbkconverter.h +++ b/sources/shiboken2/libshiboken/sbkconverter.h @@ -341,6 +341,7 @@ LIBSHIBOKEN_API bool pythonTypeIsWrapperType(const SbkConverter *converter); #define SBK_UNSIGNEDLONG_IDX 14 #define SBK_UNSIGNEDSHORT_IDX 15 #define SBK_VOIDPTR_IDX 16 +#define SBK_NULLPTR_T_IDX 17 template<typename T> SbkConverter* PrimitiveTypeConverter() { return 0; } template<> inline SbkConverter* PrimitiveTypeConverter<PY_LONG_LONG>() { return primitiveTypeConverter(SBK_PY_LONG_LONG_IDX); } @@ -360,6 +361,7 @@ template<> inline SbkConverter* PrimitiveTypeConverter<unsigned int>() { return template<> inline SbkConverter* PrimitiveTypeConverter<unsigned long>() { return primitiveTypeConverter(SBK_UNSIGNEDLONG_IDX); } template<> inline SbkConverter* PrimitiveTypeConverter<unsigned short>() { return primitiveTypeConverter(SBK_UNSIGNEDSHORT_IDX); } template<> inline SbkConverter* PrimitiveTypeConverter<void*>() { return primitiveTypeConverter(SBK_VOIDPTR_IDX); } +template<> inline SbkConverter* PrimitiveTypeConverter<std::nullptr_t>() { return primitiveTypeConverter(SBK_NULLPTR_T_IDX); } } // namespace Shiboken::Conversions @@ -386,6 +388,7 @@ template<> inline PyTypeObject* SbkType<unsigned char>() { return &PyInt_Type; } template<> inline PyTypeObject* SbkType<unsigned int>() { return &PyLong_Type; } template<> inline PyTypeObject* SbkType<unsigned long>() { return &PyLong_Type; } template<> inline PyTypeObject* SbkType<unsigned short>() { return &PyInt_Type; } +template<> inline PyTypeObject* SbkType<std::nullptr_t>() { return Py_TYPE(&_Py_NoneStruct); } } // namespace Shiboken diff --git a/sources/shiboken2/libshiboken/sbkconverter_p.h b/sources/shiboken2/libshiboken/sbkconverter_p.h index a4edfe81e..cb968ed89 100644 --- a/sources/shiboken2/libshiboken/sbkconverter_p.h +++ b/sources/shiboken2/libshiboken/sbkconverter_p.h @@ -533,6 +533,36 @@ struct Primitive<std::string> : TwoPrimitive<std::string> } }; +// nullptr_t +template <> +struct Primitive<std::nullptr_t> : TwoPrimitive<std::nullptr_t> +{ + static PyObject* toPython(const void* cppIn) + { + return Py_None; + } + static void toCpp(PyObject *, void *cppOut) + { + *reinterpret_cast<std::nullptr_t*>(cppOut) = nullptr; + } + static PythonToCppFunc isConvertible(PyObject* pyIn) + { + if (pyIn == Py_None) + return toCpp; + return nullptr; + } + static void otherToCpp(PyObject* pyIn, void* cppOut) + { + *reinterpret_cast<std::nullptr_t*>(cppOut) = nullptr; + } + static PythonToCppFunc isOtherConvertible(PyObject* pyIn) + { + if (pyIn == nullptr) + return otherToCpp; + return nullptr; + } +}; + namespace Shiboken { namespace Conversions { SbkConverter *createConverterObject(PyTypeObject *type, |