[arrow] branch master updated: ARROW-2455: [C++] Initialize the atomic bytes_allocated_ properly

[ap...@apache.org]

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

apitrou 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 c96747b  ARROW-2455: [C++] Initialize the atomic bytes_allocated_ properly
c96747b is described below

commit c96747bd0db1bb11e03ac140c91e7bb06393b22f
Author: HE, Tao <si...@gmail.com>
AuthorDate: Fri Apr 13 20:35:54 2018 +0200

    ARROW-2455: [C++] Initialize the atomic bytes_allocated_ properly
    
    The atomic counter `bytes_allocated_` in `CudaContextImpl` isn't initialized, leading to failure of cuda-test on windows.
    
    Author: HE, Tao <si...@gmail.com>
    
    Closes #1892 from sighingnow/fix-initialize and squashes the following commits:
    
    9febf78 <HE, Tao> Fix clang-format lint error.
    27e4751 <HE, Tao> Initialize the atomic bytes_allocated_ properly.
---
 cpp/src/arrow/gpu/cuda_context.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/gpu/cuda_context.cc b/cpp/src/arrow/gpu/cuda_context.cc
index 909c98a..578c04a 100644
--- a/cpp/src/arrow/gpu/cuda_context.cc
+++ b/cpp/src/arrow/gpu/cuda_context.cc
@@ -40,7 +40,7 @@ struct CudaDevice {
 
 class CudaContext::CudaContextImpl {
  public:
-  CudaContextImpl() {}
+  CudaContextImpl() : bytes_allocated_(0) {}
 
   Status Init(const CudaDevice& device) {
     device_ = device;

-- 
To stop receiving notification emails like this one, please contact
apitrou@apache.org.