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/persistent_data.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/persistent_data.hpp')
| -rw-r--r-- | include/persistent_data.hpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/persistent_data.hpp b/include/persistent_data.hpp index d7230cbf1c..47bb8a9f4c 100644 --- a/include/persistent_data.hpp +++ b/include/persistent_data.hpp @@ -43,6 +43,11 @@ class ConfigFile } } + ConfigFile(const ConfigFile&) = delete; + ConfigFile(ConfigFile&&) = delete; + ConfigFile& operator=(const ConfigFile&) = delete; + ConfigFile& operator=(ConfigFile&&) = delete; + // TODO(ed) this should really use protobuf, or some other serialization // library, but adding another dependency is somewhat outside the scope of // this application for the moment |