Skip to content

Commit

Permalink
Coding rules
Browse files Browse the repository at this point in the history
  • Loading branch information
Alex-PLACET committed Jun 7, 2024
1 parent 7daa4db commit 8259315
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
6 changes: 3 additions & 3 deletions include/sparrow/c_interface.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ namespace sparrow
return schema;
};

struct ArrowArrayCustomDeleter
struct arrow_array_custom_deleter
{
void operator()(ArrowArray* array) const
{
Expand Down Expand Up @@ -416,7 +416,7 @@ namespace sparrow
*/
template <class T, template <typename> class Allocator, std::ranges::input_range R>
requires sparrow::allocator<Allocator<T>> && std::is_integral_v<std::ranges::range_value_t<R>>
std::unique_ptr<ArrowArray, ArrowArrayCustomDeleter> make_arrow_array(
std::unique_ptr<ArrowArray, arrow_array_custom_deleter> make_arrow_array(
int64_t length,
int64_t null_count,
int64_t offset,
Expand All @@ -436,7 +436,7 @@ namespace sparrow
}
))

std::unique_ptr<ArrowArray, ArrowArrayCustomDeleter> array(new ArrowArray());
std::unique_ptr<ArrowArray, arrow_array_custom_deleter> array(new ArrowArray());

array->private_data = new arrow_array_private_data<T, Allocator>(
std::move(children),
Expand Down
3 changes: 2 additions & 1 deletion test/test_c_data_interface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,16 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#include <array>
#include <memory>
#include <optional>
#include <string_view>
#include <array>

#include "sparrow/c_interface.hpp"

#include "doctest/doctest.h"


TEST_SUITE("C Data Interface")
{
TEST_CASE("ArrowArray")
Expand Down

0 comments on commit 8259315

Please sign in to comment.