From a17df1be5e9d6cba75f939e5e7e9f52cb51395d8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Barnab=C3=A1s=20P=C5=91cze?= Date: Wed, 29 Jan 2025 15:31:27 +0100 Subject: libcamera: base: span: Explicitly default copy assignment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The `dynamic_extent` specialization is currently not trivially copyable unlike its standard counterpart, `std::span`. This is because the copy assignment operator is user-defined. Explicitly default it just like it is done in the main template definition. Signed-off-by: Barnabás Pőcze Reviewed-by: Jacopo Mondi Reviewed-by: Kieran Bingham --- include/libcamera/base/span.h | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/include/libcamera/base/span.h b/include/libcamera/base/span.h index 92cce4f0..806db106 100644 --- a/include/libcamera/base/span.h +++ b/include/libcamera/base/span.h @@ -346,13 +346,7 @@ public: } constexpr Span(const Span &other) noexcept = default; - - constexpr Span &operator=(const Span &other) noexcept - { - data_ = other.data_; - size_ = other.size_; - return *this; - } + constexpr Span &operator=(const Span &other) noexcept = default; constexpr iterator begin() const { return data(); } constexpr const_iterator cbegin() const { return begin(); } -- cgit v1.2.1