Fix: file-descriptor: unnecessary assert in move constructor

There is no use in checking the current fd in the move constructor. This
is probably a left-over from the inital move assignment operator.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I416d2778784ddea448904118c940914c030bf399

diff --git a/src/common/file-descriptor.hpp b/src/common/file-descriptor.hpp
index 51f397bcfd3c2df822b81febf26734b5bce81d31..8d6b64be4f51772fbb50692ca0577330294c7c2f 100644 (file)
--- a/src/common/file-descriptor.hpp
+++ b/src/common/file-descriptor.hpp
@@ -29,8 +29,8 @@ public:
        file_descriptor& operator=(const file_descriptor&) = delete;
        file_descriptor& operator=(file_descriptor&&) = delete;
 
-       file_descriptor(file_descriptor&& other) noexcept {
-               LTTNG_ASSERT(_is_valid_fd(_raw_fd));
+       file_descriptor(file_descriptor&& other) noexcept
+       {
                std::swap(_raw_fd, other._raw_fd);
        }