MeshTools: inline functions should not be heavy.

Returning a vector is as heavy as you can imagine.

src/Magnum/MeshTools/CombineIndexedArrays.cpp

@@ -60,6 +60,10 @@ std::pair<std::vector<UnsignedInt>, std::vector<UnsignedInt>> interleaveAndCombi
     return {combinedIndices, interleavedArrays};
 }
 
+}
+
+namespace {
+
 std::vector<UnsignedInt> combineIndexArrays(const std::reference_wrapper<std::vector<UnsignedInt>>* const begin, const std::reference_wrapper<std::vector<UnsignedInt>>* const end) {
     /* Interleave and combine the arrays */
     std::vector<UnsignedInt> combinedIndices;
@@ -83,10 +87,6 @@ std::vector<UnsignedInt> combineIndexArrays(const std::reference_wrapper<std::ve
     return combinedIndices;
 }
 
-}
-
-namespace {
-
 class IndexHash {
     public:
         explicit IndexHash(const std::vector<UnsignedInt>& indices, UnsignedInt stride): indices(indices), stride(stride) {}
@@ -115,6 +115,14 @@ class IndexEqual {
 
 }
 
+std::vector<UnsignedInt> combineIndexArrays(const std::vector<std::reference_wrapper<std::vector<UnsignedInt>>>& arrays) {
+    return combineIndexArrays(&arrays[0], &arrays[0] + arrays.size());
+}
+
+std::vector<UnsignedInt> combineIndexArrays(std::initializer_list<std::reference_wrapper<std::vector<UnsignedInt>>> arrays) {
+    return combineIndexArrays(arrays.begin(), arrays.end());
+}
+
 std::pair<std::vector<UnsignedInt>, std::vector<UnsignedInt>> combineIndexArrays(const std::vector<UnsignedInt>& interleavedArrays, const UnsignedInt stride) {
     CORRADE_ASSERT(stride != 0, "MeshTools::combineIndexArrays(): stride can't be zero", {});
     CORRADE_ASSERT(interleavedArrays.size() % stride == 0, "MeshTools::combineIndexArrays(): array size is not divisible by stride", {});

src/Magnum/MeshTools/CombineIndexedArrays.h

@@ -39,10 +39,6 @@
 
 namespace Magnum { namespace MeshTools {
 
-namespace Implementation {
-    MAGNUM_MESHTOOLS_EXPORT std::vector<UnsignedInt> combineIndexArrays(const std::reference_wrapper<std::vector<UnsignedInt>>* begin, const std::reference_wrapper<std::vector<UnsignedInt>>* end);
-}
-
 /**
 @brief Combine index arrays
 @param[in,out] arrays   Index arrays to combine. These arrays are updated
@@ -85,14 +81,10 @@ This function calls @ref combineIndexArrays(const std::vector<UnsignedInt>&, Uns
 internally. See also @ref combineIndexedArrays() which does the vertex data
 reordering automatically.
 */
-inline std::vector<UnsignedInt> combineIndexArrays(const std::vector<std::reference_wrapper<std::vector<UnsignedInt>>>& arrays) {
+MAGNUM_MESHTOOLS_EXPORT std::vector<UnsignedInt> combineIndexArrays(const std::vector<std::reference_wrapper<std::vector<UnsignedInt>>>& arrays);
-    return Implementation::combineIndexArrays(&arrays[0], &arrays[0] + arrays.size());
-}
 
 /** @overload */
-inline std::vector<UnsignedInt> combineIndexArrays(std::initializer_list<std::reference_wrapper<std::vector<UnsignedInt>>> arrays) {
+MAGNUM_MESHTOOLS_EXPORT std::vector<UnsignedInt> combineIndexArrays(std::initializer_list<std::reference_wrapper<std::vector<UnsignedInt>>> arrays);
-    return Implementation::combineIndexArrays(arrays.begin(), arrays.end());
-}
 
 /**
 @brief Combine interleaved index arrays