diff options
author | Ed Tanous <edtanous@google.com> | 2022-01-07 20:18:40 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-01-12 22:00:37 +0300 |
commit | ecd6a3a20ab128ae9d3e356d6250695800dc13be (patch) | |
tree | 658b040f57c6a4e3ee31b567606551f598a7b540 /include/ibm/locks.hpp | |
parent | 543f44000a992870ff76e76888dd589a3a31ed4e (diff) | |
download | bmcweb-ecd6a3a20ab128ae9d3e356d6250695800dc13be.tar.xz |
Enable cppcoreguidelines-special-member-functions checks
Part of enforcing cpp core guidelines involves explicitly including all
constructors required on a non-trivial class. We were missing quite a
few. In all cases, the copy/move/and operator= methods are simply
deleted.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ie8d6e8bf2bc311fa21a9ae48b0d61ee5c1940999
Diffstat (limited to 'include/ibm/locks.hpp')
-rw-r--r-- | include/ibm/locks.hpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/include/ibm/locks.hpp b/include/ibm/locks.hpp index fb8b396821..8e04b8424b 100644 --- a/include/ibm/locks.hpp +++ b/include/ibm/locks.hpp @@ -128,6 +128,14 @@ class Lock public: /* + * Explicitly deleted copy and move constructors + */ + Lock(const Lock&) = delete; + Lock(Lock&&) = delete; + Lock& operator=(const Lock&) = delete; + Lock& operator=(Lock&&) = delete; + + /* * This function implements the logic for acquiring a lock on a * resource if the incoming request is legitimate without any * conflicting requirements & without any conflicting requirement |