You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ko...@apache.org on 2021/05/25 01:56:34 UTC
[arrow] branch master updated: ARROW-12864: [C++] Remove needless
out argument from arrow::internal::InvertBitmap
This is an automated email from the ASF dual-hosted git repository.
kou 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 233a76c ARROW-12864: [C++] Remove needless out argument from arrow::internal::InvertBitmap
233a76c is described below
commit 233a76cb3392483424ba54b06d7e7e1fd1780582
Author: niranda perera <ni...@gmail.com>
AuthorDate: Tue May 25 10:54:29 2021 +0900
ARROW-12864: [C++] Remove needless out argument from arrow::internal::InvertBitmap
We should have removed the out argument in b218a7fdae0792e185579d8cd20748ed0752b9ff.
Closes #10391 from nirandaperera/ARROW-12864
Authored-by: niranda perera <ni...@gmail.com>
Signed-off-by: Sutou Kouhei <ko...@clear-code.com>
---
cpp/src/arrow/util/bitmap_ops.cc | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/cpp/src/arrow/util/bitmap_ops.cc b/cpp/src/arrow/util/bitmap_ops.cc
index 1f9cf19..32da60a 100644
--- a/cpp/src/arrow/util/bitmap_ops.cc
+++ b/cpp/src/arrow/util/bitmap_ops.cc
@@ -390,8 +390,7 @@ Result<std::shared_ptr<Buffer>> CopyBitmap(MemoryPool* pool, const uint8_t* data
}
Result<std::shared_ptr<Buffer>> InvertBitmap(MemoryPool* pool, const uint8_t* data,
- int64_t offset, int64_t length,
- std::shared_ptr<Buffer>* out) {
+ int64_t offset, int64_t length) {
return TransferBitmap<TransferMode::Invert>(pool, data, offset, length);
}