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/vm_websocket.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/vm_websocket.hpp')
-rw-r--r-- | include/vm_websocket.hpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/include/vm_websocket.hpp b/include/vm_websocket.hpp index 34ef82ef99..2a0353be32 100644 --- a/include/vm_websocket.hpp +++ b/include/vm_websocket.hpp @@ -32,6 +32,11 @@ class Handler : public std::enable_shared_from_this<Handler> ~Handler() = default; + Handler(const Handler&) = delete; + Handler(Handler&&) = delete; + Handler& operator=(const Handler&) = delete; + Handler& operator=(Handler&&) = delete; + void doClose() { // boost::process::child::terminate uses SIGKILL, need to send SIGTERM |