Use more efficient implementation of get_new_inst_path

Co-authored-by: Kenneth Loeffler <[email protected]>

src/syncback/snapshot.rs

@@ -128,24 +128,17 @@ impl<'sync> SyncbackSnapshot<'sync> {
     /// where you would look for the object in Roblox Studio.
     #[inline]
     pub fn get_new_inst_path(&self, referent: Ref) -> String {
-        let mut path = Vec::new();
-        let mut path_capacity = 0;
-
-        let mut inst = self.get_new_instance(referent);
-        while let Some(instance) = inst {
-            path.push(&instance.name);
-            path_capacity += instance.name.len() + 1;
-            inst = self.get_new_instance(instance.parent());
-        }
-        let mut str = String::with_capacity(path_capacity);
-        while let Some(segment) = path.pop() {
-            str.push_str(segment);
-            str.push('/')
-        }
-        str.pop();
+       let mut path = Vec::new();
 
-        str
-    }
+       let mut inst = self.get_new_instance(referent);
+       while let Some(instance) = inst {
+           path.push(instance.name.as_str());
+           inst = self.get_new_instance(instance.parent());
+       }
+
+       path.reverse();
+       path.join("/")
+   }
 
     /// Returns an Instance from the old tree with the provided referent, if it
     /// exists.