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 /http/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 'http/websocket.hpp')
| -rw-r--r-- | http/websocket.hpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/http/websocket.hpp b/http/websocket.hpp index 30a9b9f4b5..74bce581d2 100644 --- a/http/websocket.hpp +++ b/http/websocket.hpp @@ -29,6 +29,11 @@ struct Connection : std::enable_shared_from_this<Connection> req(reqIn.req), userName{std::move(user)}, userdataPtr(nullptr) {} + Connection(const Connection&) = delete; + Connection(Connection&&) = delete; + Connection& operator=(const Connection&) = delete; + Connection& operator=(const Connection&&) = delete; + virtual void sendBinary(const std::string_view msg) = 0; virtual void sendBinary(std::string&& msg) = 0; virtual void sendText(const std::string_view msg) = 0; |