You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by fs...@apache.org on 2020/04/23 00:42:21 UTC

[arrow] branch master updated: ARROW-8554 [C++][Benchmark] Fix building error "cannot bind lvalue"

This is an automated email from the ASF dual-hosted git repository.

fsaintjacques pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new fc0ed86  ARROW-8554 [C++][Benchmark] Fix building error "cannot bind lvalue"
fc0ed86 is described below

commit fc0ed8681ce5703a6ed98ca05ccbdbcfe2269c1d
Author: Li, Jiajia <ji...@intel.com>
AuthorDate: Wed Apr 22 20:41:44 2020 -0400

    ARROW-8554 [C++][Benchmark] Fix building error "cannot bind lvalue"
    
    https://issues.apache.org/jira/browse/ARROW-8554
    
    Closes #7011 from plusplusjiajia/fix-bitutilbenchmark
    
    Authored-by: Li, Jiajia <ji...@intel.com>
    Signed-off-by: François Saint-Jacques <fs...@gmail.com>
---
 cpp/src/arrow/util/bit_util_benchmark.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/util/bit_util_benchmark.cc b/cpp/src/arrow/util/bit_util_benchmark.cc
index dc156fb..fd8855c 100644
--- a/cpp/src/arrow/util/bit_util_benchmark.cc
+++ b/cpp/src/arrow/util/bit_util_benchmark.cc
@@ -93,7 +93,7 @@ static std::shared_ptr<Buffer> CreateRandomBuffer(int64_t nbytes) {
   auto buffer = *AllocateBuffer(nbytes);
   memset(buffer->mutable_data(), 0, nbytes);
   random_bytes(nbytes, 0, buffer->mutable_data());
-  return buffer;
+  return std::move(buffer);
 }
 
 template <typename DoAnd>