diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-08-31 11:07:55 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-09-05 06:55:16 +0000 |
commit | 72f399ce08c58e955e94185200d70f0088cc1ff0 (patch) | |
tree | bdb5193b8f2167840b76599e12fd7d399ef2b17b /sources/pyside2/libpyside/signalmanager.h | |
parent | 30182c1bb7112b61b8bc29801c9ced6078e715a2 (diff) |
libpyside/PyObjectWrapper: Fix clang-tidy warnings about class definition
Rearrange definition and delete move constructor and move assignment operator.
This unearthed that the (default-generated) move assignment operator
was used (unintendedly) in:
QDataStream &operator>>(QDataStream&,PyObjectWrapper&).
Factor out a reset() method (modeled after std::unique_ptr, QScopedPointer)
for this.
Change-Id: I48c4f9ca1353984da870bb7eb3308a9f3374d309
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Diffstat (limited to 'sources/pyside2/libpyside/signalmanager.h')
-rw-r--r-- | sources/pyside2/libpyside/signalmanager.h | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/sources/pyside2/libpyside/signalmanager.h b/sources/pyside2/libpyside/signalmanager.h index c4d57c333..745c1e55a 100644 --- a/sources/pyside2/libpyside/signalmanager.h +++ b/sources/pyside2/libpyside/signalmanager.h @@ -53,12 +53,19 @@ namespace PySide class PYSIDE_API PyObjectWrapper { public: + PyObjectWrapper(PyObjectWrapper&&) = delete; + PyObjectWrapper& operator=(PyObjectWrapper &&) = delete; + PyObjectWrapper(); explicit PyObjectWrapper(PyObject* me); PyObjectWrapper(const PyObjectWrapper &other); + PyObjectWrapper& operator=(const PyObjectWrapper &other); + + void reset(PyObject *o); + ~PyObjectWrapper(); operator PyObject*() const; - PyObjectWrapper& operator=(const PyObjectWrapper &other); + private: PyObject* m_me; }; |