From aa62b71bb35c6dfddf79ae63acf7644df1d98a72 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sun, 8 Aug 2021 21:41:34 +0800 Subject: [PATCH] test/crimson: drop copy assignment operator because of the const reference member variable of `parent`: const delta_overlay_t &parent; we are not allowed to mutate it. so the explicitly defaulted copy assignment operator is implicitly deleted this change silences the warnings from Clang like: ../src/test/crimson/seastore/test_transaction_manager.cc:159:12: warning: explicitly defaulted copy assignment operator is implicitly deleted [-Wdefaulted-function-deleted] iterator &operator=(const iterator &) = default; ^ ../src/test/crimson/seastore/test_transaction_manager.cc:98:25: note: copy assignment operator of 'iterator' is implicitly deleted because field 'parent' is of reference type 'const transaction_manager_$ const delta_overlay_t &parent; ^ Signed-off-by: Kefu Chai --- src/test/crimson/seastore/test_transaction_manager.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/test/crimson/seastore/test_transaction_manager.cc b/src/test/crimson/seastore/test_transaction_manager.cc index db93baaeb673f..523934ed2ed24 100644 --- a/src/test/crimson/seastore/test_transaction_manager.cc +++ b/src/test/crimson/seastore/test_transaction_manager.cc @@ -156,8 +156,6 @@ struct transaction_manager_test_t : public: iterator(const iterator &) = default; iterator(iterator &&) = default; - iterator &operator=(const iterator &) = default; - iterator &operator=(iterator &&) = default; iterator &operator++() { assert(is_valid()); -- 2.39.5