FIX: Add missing forward of forwarding references

RUrlus · Oct 23, 2023 · 936427d · 936427d
1 parent 2180a1b
commit 936427d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/include/carma_bits/base/converters.hpp b/include/carma_bits/base/converters.hpp
@@ -3,6 +3,7 @@
 #include <pybind11/pybind11.h>
 #include <armadillo>
 #include <carma_bits/base/numpy_converters.hpp>
+#include <utility>
 
 namespace py = pybind11;
 
@@ -51,7 +52,7 @@ auto arr_to_row(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_row(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Row<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Row<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /*******************************************************************************
@@ -98,7 +99,7 @@ auto arr_to_col(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_col(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Col<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Col<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /*******************************************************************************
@@ -145,7 +146,7 @@ auto arr_to_mat(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_mat(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Mat<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Mat<eT>>()(std::forward<decltype(arr)>(arr));
 }
 /*******************************************************************************
  *                               ARR_TO_CUBE                                   *
@@ -191,6 +192,6 @@ auto arr_to_cube(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_cube(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Cube<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Cube<eT>>()(std::forward<decltype(arr)>(arr));
 }
 }  // namespace carma
diff --git a/include/carma_bits/extension/converters.hpp b/include/carma_bits/extension/converters.hpp
@@ -162,7 +162,7 @@ auto arr_to_row(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_row(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Row<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Row<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /**
@@ -224,7 +224,7 @@ auto arr_to_col(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_col(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Col<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Col<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /**
@@ -286,7 +286,7 @@ auto arr_to_mat(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_mat(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Mat<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Mat<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /**
@@ -348,7 +348,7 @@ auto arr_to_cube(const py::array_t<eT>& arr) {
  */
 template <typename eT>
 auto arr_to_cube(py::array_t<eT>&& arr) {
-    return internal::DefaultNumpyConverter<arma::Cube<eT>>()(arr);
+    return internal::DefaultNumpyConverter<arma::Cube<eT>>()(std::forward<decltype(arr)>(arr));
 }
 
 /**