diff options
| author | Radoslaw Zarzynski <rzarzyns@redhat.com> | 2020-06-23 12:28:15 +0200 |
|---|---|---|
| committer | Radoslaw Zarzynski <rzarzyns@redhat.com> | 2020-06-29 13:29:54 +0200 |
| commit | 3fdfd606f1344317931207e3d3c123d62acb673b (patch) | |
| tree | 44bfc8eb15db70d72ba9f6b0f3eb0d38e4321082 /src/ceph_mon.cc | |
| parent | Merge pull request #35635 from zdover23/wip-doc-man-deploy-mon-bootstrapping-... (diff) | |
| download | ceph-3fdfd606f1344317931207e3d3c123d62acb673b.tar.xz ceph-3fdfd606f1344317931207e3d3c123d62acb673b.zip | |
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 <rzarzyns@redhat.com>
Diffstat (limited to 'src/ceph_mon.cc')
| -rw-r--r-- | src/ceph_mon.cc | 2 |
1 files changed, 1 insertions, 1 deletions
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; |