Fix lint, address code review comments

apache · Oct 7, 2024 · a5d9e2d · a5d9e2d
1 parent 56c394d
commit a5d9e2d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/cpp/src/arrow/array/data.h b/cpp/src/arrow/array/data.h
@@ -17,7 +17,6 @@
 
 #pragma once
 
-#include <arrow/util/range.h>
 #include <atomic>  // IWYU pragma: export
 #include <cassert>
 #include <cstdint>
@@ -32,6 +31,7 @@
 #include "arrow/type_fwd.h"
 #include "arrow/util/bit_util.h"
 #include "arrow/util/macros.h"
+#include "arrow/util/range.h"
 #include "arrow/util/span.h"
 #include "arrow/util/visibility.h"
 
@@ -242,8 +242,8 @@ struct ARROW_EXPORT ArrayData {
       }
     }
     // align children data recursively
-    for (unsigned int i = 0; i < child_data.size(); i++) {
-      RETURN_NOT_OK(child_data[i]->AlignBuffers());
+    for (const auto& child : child_data) {
+      RETURN_NOT_OK(child->AlignBuffers());
     }
     return Status::OK();
   }

diff --git a/python/pyarrow/tests/test_ipc.py b/python/pyarrow/tests/test_ipc.py
@@ -549,7 +549,6 @@ def test_read_options():
     assert options.use_threads is True
     assert options.ensure_native_endian is True
     assert options.ensure_memory_alignment is True
-    assert options.ens is True
     assert options.included_fields == []
 
     options.ensure_native_endian = False