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>

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;