GH-41024: [C++] IO: fixing compiling in gcc 7.5.0 (#41025)

mapleFU · web-flow · commit 110efed4d0d8 · 2024-04-05T21:01:52.000+09:00
### Rationale for this change Previous pr ( #39807 ) remove std::move when returning value, however, it's not allowed in some old compilers ### What changes are included in this PR? add std::move for return, and add reason for that ### Are these changes tested? Should test by other ci ### Are there any user-facing changes? no * GitHub Issue: #41024 Authored-by: mwish <maplewish117@gmail.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
diff --git a/cpp/src/arrow/io/compressed.cc b/cpp/src/arrow/io/compressed.cc
@@ -405,7 +405,9 @@ class CompressedInputStream::Impl {
     ARROW_ASSIGN_OR_RAISE(auto buf, AllocateResizableBuffer(nbytes, pool_));
     ARROW_ASSIGN_OR_RAISE(int64_t bytes_read, Read(nbytes, buf->mutable_data()));
     RETURN_NOT_OK(buf->Resize(bytes_read));
-    return buf;
+    // Using std::move because the some compiler might has issue below:
+    // https://wg21.cmeerw.net/cwg/issue1579
+    return std::move(buf);
   }
 
   const std::shared_ptr<InputStream>& raw() const { return raw_; }
