Skip to content

Commit

Permalink
Fix linter errors
Browse files Browse the repository at this point in the history
  • Loading branch information
sgilmore10 committed Sep 5, 2023
1 parent b82e30c commit 2973bda
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion cpp/src/arrow/chunked_array.cc
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ namespace {
// Only floating types support NaN
supports_nan |= is_floating(type.id());
} else {
for(const auto& field : type.fields()) {
for (const auto& field : type.fields()) {
supports_nan |= supportsNaN(*field->type());
if (supports_nan) {
break;
Expand Down
4 changes: 2 additions & 2 deletions cpp/src/arrow/chunked_array_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ TEST_F(TestChunkedArray, EqualsSameAddressWithNaNs) {
ASSERT_OK_AND_ASSIGN(auto result0, ChunkedArray::Make({chunk0, chunk1}, float64()));
// result0 has NaN values
ASSERT_FALSE(result0->Equals(result0));

auto chunk2 = ArrayFromJSON(float64(), "[6, 7, 8, 9]");
ASSERT_OK_AND_ASSIGN(auto result1, ChunkedArray::Make({chunk1, chunk2}, float64()));
// result1 does not have NaN values
Expand All @@ -165,7 +165,7 @@ TEST_F(TestChunkedArray, EqualsSameAddressWithNaNs) {
ASSERT_OK_AND_ASSIGN(auto result2, ChunkedArray::Make({struct0}, struct0->type()));
// result2 has NaN values
ASSERT_FALSE(result2->Equals(result2));

auto array2 = ArrayFromJSON(float64(), "[0, 1, 2]");
ASSERT_OK_AND_ASSIGN(auto struct1, StructArray::Make({array0, array2}, fieldnames));
ASSERT_OK_AND_ASSIGN(auto result3, ChunkedArray::Make({struct1}, struct1->type()));
Expand Down

0 comments on commit 2973bda

Please sign in to comment.