crimson/os/seastore: remove CachedExtent move constructor

I'm not sure why I added this in the first place, there aren't really
any scenarios where invoking it wouldn't be a bug.

Signed-off-by: Samuel Just <sjust@redhat.com>
This commit is contained in:
Samuel Just 2020-08-12 11:10:56 -07:00
parent 5f2bd86f21
commit 7670fccc42
2 changed files with 2 additions and 5 deletions

View File

@ -313,7 +313,7 @@ private:
}
protected:
CachedExtent(CachedExtent &&other) = default;
CachedExtent(CachedExtent &&other) = delete;
CachedExtent(ceph::bufferptr &&ptr) : ptr(std::move(ptr)) {}
CachedExtent(const CachedExtent &other)
: state(other.state),

View File

@ -15,10 +15,7 @@ struct OnodeBlock final : LogicalCachedExtent {
template <typename... T>
OnodeBlock(T&&... t) : LogicalCachedExtent(std::forward<T>(t)...) {}
OnodeBlock(OnodeBlock&& block) noexcept
: LogicalCachedExtent{std::move(block)},
deltas{std::move(block.deltas)}
{}
OnodeBlock(OnodeBlock&& block) = delete;
OnodeBlock(const OnodeBlock& block, CachedExtent::share_buffer_t tag) noexcept
: LogicalCachedExtent{block, tag},
share_buffer{true}