*: switch trivial users of bufferlist::copy{_in} to iterators.

This commits switches the most straightforward occurrences of:
  * ceph::bufferlist::copy(off, len, dst) and
  * ceph::bufferlist::copy_in(off, len, src)
to use iterators. After extending `bufferlist::begin()` with
the `offset` parameter and eradication of the `last_p` hint,
these method look rudimentary. There is no huge benefit from

  bl.copy_in(off, len, dst)

instead of:

  bl.begin(off).copy_in(len, dst)

especially when `off` is `0`.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

1 files changed, 1 insertions, 1 deletions

diff --git a/src/crush/CrushLocation.cc b/src/crush/CrushLocation.cc
index 0e5a0855cd6..af940bc9f7c 100644
--- a/src/crush/CrushLocation.cc
+++ b/src/crush/CrushLocation.cc
@@ -89,7 +89,7 @@ int CrushLocation::update_from_hook()
     return ret;
 
   std::string out;
-  bl.copy(0, bl.length(), out);
+  bl.begin().copy(bl.length(), out);
   out.erase(out.find_last_not_of(" \n\r\t")+1);
   return _parse(out);
 }