Make 'apply_permutation' function public

Author: eranfu

soa-derive-internal/src/slice.rs

@@ -341,7 +341,7 @@ pub fn derive_mut(input: &Input) -> TokenStream {
     nested_ord.push(quote! { for<'b> #ref_name<'b>: Ord });
 
     let apply_permutation = input.map_fields_nested_or(
-        |ident, _| quote! { self.#ident.apply_permutation(permutation) },
+        |ident, _| quote! { self.#ident.__private_apply_permutation(permutation) },
         |ident, _| quote! { permutation.apply_slice_in_place(&mut self.#ident) },
     ).collect::<Vec<_>>();
 
@@ -616,7 +616,9 @@ pub fn derive_mut(input: &Input) -> TokenStream {
             }
 
             #[doc(hidden)]
-            fn apply_permutation(&mut self, permutation: &mut soa_derive::Permutation) {
+            /// This is `pub` due to there will be compile-error if `#[nested_soa]` is used.
+            /// Do not use this method directly.
+            pub fn __private_apply_permutation(&mut self, permutation: &mut soa_derive::Permutation) {
                 #( #apply_permutation; )*
             }
 
@@ -633,7 +635,7 @@ pub fn derive_mut(input: &Input) -> TokenStream {
                 permutation.sort_by(|j, k| f(self.index(*j), self.index(*k)));
 
                 let mut permutation = Permutation::oneline(permutation).inverse();
-                self.apply_permutation(&mut permutation);
+                self.__private_apply_permutation(&mut permutation);
             }
 
             /// Similar to [`&mut
@@ -650,7 +652,7 @@ pub fn derive_mut(input: &Input) -> TokenStream {
                 permutation.sort_by_key(|i| f(self.index(*i)));
 
                 let mut permutation = Permutation::oneline(permutation).inverse();
-                self.apply_permutation(&mut permutation);
+                self.__private_apply_permutation(&mut permutation);
             }
         }
 
@@ -669,7 +671,7 @@ pub fn derive_mut(input: &Input) -> TokenStream {
                 permutation.sort_by_key(|i| self.index(*i));
 
                 let mut permutation = Permutation::oneline(permutation).inverse();
-                self.apply_permutation(&mut permutation);
+                self.__private_apply_permutation(&mut permutation);
             }
         }
     };

tests/nested_soa.rs

@@ -6,15 +6,21 @@ pub struct Point {
     y: f32,
 }
 
-#[derive(Debug, Clone, PartialEq, StructOfArray)]
-#[soa_derive(Debug, Clone, PartialEq)]
-pub struct Color {
-    pub r: u8,
-    pub g: u8,
-    pub b: u8,
-    pub a: u8,
+mod other_mod {
+    use soa_derive::StructOfArray;
+
+    #[derive(Debug, Clone, PartialEq, StructOfArray)]
+    #[soa_derive(Debug, Clone, PartialEq)]
+    pub struct Color {
+        pub r: u8,
+        pub g: u8,
+        pub b: u8,
+        pub a: u8,
+    }
 }
 
+use other_mod::*;
+
 #[derive(Debug, Clone, PartialEq, StructOfArray)]
 #[soa_derive(Debug, Clone, PartialEq)]
 pub struct Particle {