Remove unnecessary construction of IOBuf in Cursor

Summary: This assignment is unnecessary, because in `for` loop we **always** call `cloneOneInto` that would assign another IOBuf to that IOBuf.

Reviewed By: alikhtarov

Differential Revision: D3215861

fb-gh-sync-id: ad87b99848aaae79f0870d49e0474f1abf0f28e5
fbshipit-source-id: ad87b99848aaae79f0870d49e0474f1abf0f28e5

diff --git a/folly/io/Cursor.h b/folly/io/Cursor.h
index 74412ea6720cdb95c6aa6830e0dff4a464de40b5..3cf369e9ece27298ffa7eb2bf35a225bc83152a8 100644 (file)
--- a/folly/io/Cursor.h
+++ b/folly/io/Cursor.h
@@ -324,8 +324,6 @@ class CursorBase {
   }
 
   size_t cloneAtMost(folly::IOBuf& buf, size_t len) {
-    buf = folly::IOBuf();
-
     std::unique_ptr<folly::IOBuf> tmp;
     size_t copied = 0;
     for (int loopCount = 0; true; ++loopCount) {

diff --git a/folly/io/test/IOBufCursorBenchmark.cpp b/folly/io/test/IOBufCursorBenchmark.cpp
index e7142d9c214cfc84cdca3d474eb84991ff9bd4e8..233f800b2921bb0a5c4e5f1d60edbc091a87f9c8 100644 (file)
--- a/folly/io/test/IOBufCursorBenchmark.cpp
+++ b/folly/io/test/IOBufCursorBenchmark.cpp
@@ -76,6 +76,16 @@ BENCHMARK(skipBenchmark, iters) {
   }
 }
 
+BENCHMARK(cloneBenchmark, iters) {
+  folly::IOBuf out;
+  while (iters--) {
+    Cursor c(iobuf_read_benchmark.get());
+    for (int i = 0; i < benchmark_size; ++i) {
+      c.clone(out, 1);
+    }
+  }
+}
+
 // fbmake opt
 // _bin/folly/experimental/io/test/iobuf_cursor_test -benchmark
 //