You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by li...@apache.org on 2022/05/24 13:34:31 UTC

[arrow] branch master updated: MINOR: [C++] Mark ChunkResolver move constructor noexcept

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

lidavidm 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 dd84c0f68c MINOR: [C++] Mark ChunkResolver move constructor noexcept
dd84c0f68c is described below

commit dd84c0f68c6f898e3a02bb0623500e3f165f80d4
Author: Yibo Cai <yi...@arm.com>
AuthorDate: Tue May 24 09:34:12 2022 -0400

    MINOR: [C++] Mark ChunkResolver move constructor noexcept
    
    It also makes ChunkedArray nothrow move constructible as it contains a
    ChunkResolver data member.
    
    Closes #13223 from cyb70289/chunk-resolver-noexcept-move
    
    Authored-by: Yibo Cai <yi...@arm.com>
    Signed-off-by: David Li <li...@gmail.com>
---
 cpp/src/arrow/chunk_resolver.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/chunk_resolver.h b/cpp/src/arrow/chunk_resolver.h
index f69253b161..1a63d26c24 100644
--- a/cpp/src/arrow/chunk_resolver.h
+++ b/cpp/src/arrow/chunk_resolver.h
@@ -39,7 +39,7 @@ struct ChunkResolver {
 
   explicit ChunkResolver(const RecordBatchVector& batches);
 
-  ChunkResolver(ChunkResolver&& other)
+  ChunkResolver(ChunkResolver&& other) noexcept
       : offsets_(std::move(other.offsets_)), cached_chunk_(other.cached_chunk_.load()) {}
 
   ChunkResolver& operator=(ChunkResolver&& other) {