From b3467840f965a9792465c2a456a98387ff94dc27 Mon Sep 17 00:00:00 2001 From: Bram Veenboer Date: Thu, 14 Aug 2025 11:07:45 +0200 Subject: [PATCH] Use const size_t for n --- python/bindings.cpp | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/python/bindings.cpp b/python/bindings.cpp index ba63955..c8153df 100644 --- a/python/bindings.cpp +++ b/python/bindings.cpp @@ -10,7 +10,7 @@ template py::array_t compute_sin(const Backend &backend, py::array_t x) { - ssize_t n = x.shape(0); + const size_t n = x.shape(0); if (x.ndim() != 1) { throw py::value_error("Input array must be 1-dimensional"); } @@ -20,7 +20,7 @@ compute_sin(const Backend &backend, py::array_t s(n); T *s_ptr = s.mutable_data(); - backend.compute_sinf(static_cast(n), x_ptr, s_ptr); + backend.compute_sinf(n, x_ptr, s_ptr); return s; } @@ -29,7 +29,7 @@ template py::array_t compute_cos(const Backend &backend, py::array_t x) { - ssize_t n = x.shape(0); + const size_t n = x.shape(0); if (x.ndim() != 1) { throw py::value_error("Input array must be 1-dimensional"); } @@ -39,7 +39,7 @@ compute_cos(const Backend &backend, py::array_t c(n); T *c_ptr = c.mutable_data(); - backend.compute_cosf(static_cast(n), x_ptr, c_ptr); + backend.compute_cosf(n, x_ptr, c_ptr); return c; } @@ -48,7 +48,7 @@ template std::tuple, py::array_t> compute_sincos(const Backend &backend, py::array_t x) { - ssize_t n = x.shape(0); + const size_t n = x.shape(0); if (x.ndim() != 1) { throw py::value_error("Input array must be 1-dimensional"); } @@ -58,8 +58,7 @@ compute_sincos(const Backend &backend, py::array_t s(n); py::array_t c(n); - backend.compute_sincosf(static_cast(n), x_ptr, s.mutable_data(), - c.mutable_data()); + backend.compute_sincosf(n, x_ptr, s.mutable_data(), c.mutable_data()); return std::make_tuple(s, c); }