[arrow-nanoarrow] branch main updated: chore: Fix C++ warnings causing ADBC R package errors (#181)

[paleolimbot]

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

paleolimbot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-nanoarrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 4c28c7e  chore: Fix C++ warnings causing ADBC R package errors (#181)
4c28c7e is described below

commit 4c28c7e1dccc1c9863e7e7cd1d5f92644a311495
Author: Dewey Dunnington <de...@dunnington.ca>
AuthorDate: Wed Apr 19 15:56:12 2023 -0400

    chore: Fix C++ warnings causing ADBC R package errors (#181)
    
    Closes #179.
---
 src/nanoarrow/nanoarrow.hpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/nanoarrow/nanoarrow.hpp b/src/nanoarrow/nanoarrow.hpp
index 1fe25db..b01d2a6 100644
--- a/src/nanoarrow/nanoarrow.hpp
+++ b/src/nanoarrow/nanoarrow.hpp
@@ -278,10 +278,10 @@ class VectorArrayStream : public EmptyArrayStream {
 
  protected:
   VectorArrayStream(struct ArrowSchema* schema, std::vector<UniqueArray> 
arrays)
-      : EmptyArrayStream(schema), offset_(0), arrays_(std::move(arrays)) {}
+      : EmptyArrayStream(schema), arrays_(std::move(arrays)), offset_(0) {}
 
   int get_next(struct ArrowArray* array) {
-    if (offset_ < arrays_.size()) {
+    if (offset_ < static_cast<int64_t>(arrays_.size())) {
       arrays_[offset_++].move(array);
     } else {
       array->release = nullptr;