diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-08-31 10:09:39 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2018-08-31 10:06:13 +0000 |
commit | 4af51767cf8514b196ce9be76c6c31111ea5fe3f (patch) | |
tree | ff073094ee01e7f7972c30efd1a1fe32edfd2164 /sources/shiboken2/libshiboken/bindingmanager.h | |
parent | e95e2b74f109eaf167782cb916f76d72eaac9ed6 (diff) |
libshiboken: Fix clang-tidy warnings about class definitions
- Add override
- Use = default for trivial constructors/destructors
- Delete copy and move constructors/assignment and move assignment
operators where not needed
- Use member initialization and remove constructors from simple structs
- Use explicit where applicable
Change-Id: Id293dd0008b05243e665347f12fd1dee3b1b70f7
Reviewed-by: Christian Tismer <tismer@stackless.com>
Diffstat (limited to 'sources/shiboken2/libshiboken/bindingmanager.h')
-rw-r--r-- | sources/shiboken2/libshiboken/bindingmanager.h | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/sources/shiboken2/libshiboken/bindingmanager.h b/sources/shiboken2/libshiboken/bindingmanager.h index 13a4d3a2e..c09b985dc 100644 --- a/sources/shiboken2/libshiboken/bindingmanager.h +++ b/sources/shiboken2/libshiboken/bindingmanager.h @@ -55,6 +55,11 @@ typedef void (*ObjectVisitor)(SbkObject*, void*); class LIBSHIBOKEN_API BindingManager { public: + BindingManager(const BindingManager&) = delete; + BindingManager(BindingManager&&) = delete; + BindingManager& operator=(const BindingManager&) = delete; + BindingManager& operator=(BindingManager&&) = delete; + static BindingManager& instance(); bool hasWrapper(const void *cptr); @@ -94,10 +99,7 @@ public: private: ~BindingManager(); - // disable copy BindingManager(); - BindingManager(const BindingManager&); - BindingManager& operator=(const BindingManager&); struct BindingManagerPrivate; BindingManagerPrivate* m_d; |