From 3fdfd606f1344317931207e3d3c123d62acb673b Mon Sep 17 00:00:00 2001 From: Radoslaw Zarzynski Date: Tue, 23 Jun 2020 10:28:15 +0000 Subject: common/bl, *: deprecate list::claim() in favor of operator=(list&&). The motivation is that `claim(list&)` seems to actually be a pre-C++11 counterpart of the already available `operator=(list&&)`. This commit deprecates the `claim()` method but doesn't drop it yet. All occurrences of `buffer::list::claim(list&)` are switched to * `list::operator=(list&&)` or * reworked to use `list::list(list&&)` instead. Changes are applied to: rgw, osdc, osd, os/memstore, os/filestore, os/bluestore, os, msg, mgr, messages, mds, librbd, librados, crimson, common, cls, mon. Signed-off-by: Radoslaw Zarzynski --- src/ceph_mon.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/ceph_mon.cc') diff --git a/src/ceph_mon.cc b/src/ceph_mon.cc index 5e35b61e7e9..3bd219f1107 100644 --- a/src/ceph_mon.cc +++ b/src/ceph_mon.cc @@ -101,7 +101,7 @@ int obtain_monmap(MonitorDBStore &store, bufferlist &bl) if (b.get_epoch() > latest_ver) { dout(10) << __func__ << " using stashed monmap " << b.get_epoch() << " instead" << dendl; - bl.claim(bl2); + bl = std::move(bl2); } else { dout(10) << __func__ << " ignoring stashed monmap " << b.get_epoch() << dendl; -- cgit v1.2.3