[arrow-nanoarrow] branch main updated: fix: Fix declaration of an array with an ambiguously constexpr size (#301)

[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 faacb1c  fix: Fix declaration of an array with an ambiguously 
constexpr size (#301)
faacb1c is described below

commit faacb1cc50814ed616d79d67bfcae9fd1d8f286d
Author: Dewey Dunnington <de...@dunnington.ca>
AuthorDate: Thu Sep 21 23:05:37 2023 +0200

    fix: Fix declaration of an array with an ambiguously constexpr size (#301)
    
    Closes #283.
---
 src/nanoarrow/buffer_test.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/nanoarrow/buffer_test.cc b/src/nanoarrow/buffer_test.cc
index 7ca4874..de8741c 100644
--- a/src/nanoarrow/buffer_test.cc
+++ b/src/nanoarrow/buffer_test.cc
@@ -293,8 +293,8 @@ void TestArrowBitmapUnpackUnsafe(const uint8_t* bitmap, 
std::vector<int8_t> expe
 
 TEST(BitmapTest, BitmapTestBitmapUnpack) {
   uint8_t bitmap[3];
-  int64_t n_values = sizeof(bitmap) * 8;
-  int8_t result[n_values];
+  int8_t result[sizeof(bitmap) * 8];
+  int64_t n_values = sizeof(result);
   int32_t result32[n_values];
 
   // Basic test of a validity buffer that is all true