This is an automated email from the ASF dual-hosted git repository.
wesm 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 4fe30d5 ARROW-3516: [C++] Use unsigned type for difference of
pointers in parallel_memcpy
4fe30d5 is described below
commit 4fe30d5306a7090831c446eb8035cb370d3ec9b2
Author: Kouhei Sutou <[email protected]>
AuthorDate: Mon Oct 15 09:46:20 2018 -0400
ARROW-3516: [C++] Use unsigned type for difference of pointers in
parallel_memcpy
Author: Kouhei Sutou <[email protected]>
Closes #2760 from kou/cpp-fix-memcpy-size-type and squashes the following
commits:
14c59bad2 <Kouhei Sutou> Fix memcpy's size type
---
cpp/src/arrow/util/memory.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/src/arrow/util/memory.h b/cpp/src/arrow/util/memory.h
index 121f301..eac6cc4 100644
--- a/cpp/src/arrow/util/memory.h
+++ b/cpp/src/arrow/util/memory.h
@@ -47,7 +47,7 @@ void parallel_memcopy(uint8_t* dst, const uint8_t* src,
int64_t nbytes,
// Now we divide these blocks between available threads. The remainder is
// handled separately.
- int64_t chunk_size = (right - left) / num_threads;
+ size_t chunk_size = (right - left) / num_threads;
int64_t prefix = left - src;
int64_t suffix = src + nbytes - right;
// Now the data layout is | prefix | k * num_threads * block_size | suffix |.
