Don't destroy socket by default as they will always be open.

Now derivated class use constructor from base class and we don't destroy
socket when a socket object is over since it may be copied before. So closing
them is in charge of dev, well me...

Change-Id: I0440119017a3e56bb83d0194a6908dc3e2b8f745
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>

1 files changed, 6 insertions, 5 deletions

diff --git a/CAN-binder/low-can-binding/utils/socketcan.cpp b/CAN-binder/low-can-binding/utils/socketcan.cpp
index 4d22654..f52aaa6 100644
--- a/CAN-binder/low-can-binding/utils/socketcan.cpp
+++ b/CAN-binder/low-can-binding/utils/socketcan.cpp
@@ -31,6 +31,11 @@ namespace utils
 		: socket_{INVALID_SOCKET}
 	{}
 
+	/// @brief Construct a socket by copying an existing one.
+	socketcan_t::socketcan_t(const socketcan_t& s)
+		: socket_{s.socket_}
+	{}
+
 	/// @brief Construct a socket by moving an existing one.
 	socketcan_t::socketcan_t(socketcan_t&& s)
 		: socket_{s.socket_}
@@ -38,12 +43,8 @@ namespace utils
 		s.socket_ = INVALID_SOCKET;
 	}
 
-	/// @brief Destruct the socket.
 	socketcan_t::~socketcan_t()
-	{
-		if(socket_ != INVALID_SOCKET)
-			::close(socket_);
-	}
+	{}
 
 	const struct sockaddr_can& socketcan_t::get_tx_address() const
 	{