Merge pull request #498 from nekobako/remove-unused-bone-references

chore: remove unused bone references just after reading mesh
anatawa12 · Sep 23, 2023 · 88cf2fd · 88cf2fd
2 parents def7d18 + fc31517
commit 88cf2fd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/CHANGELOG-PRERELEASE.md b/CHANGELOG-PRERELEASE.md
@@ -16,6 +16,7 @@ The format is based on [Keep a Changelog].
 ### Removed
 
 ### Fixed
+- Remove unused bone references [`#498`](https://github.com/anatawa12/AvatarOptimizer/pull/498)
 
 ### Security
 

diff --git a/Editor/Processors/SkinnedMeshes/MeshInfo2.cs b/Editor/Processors/SkinnedMeshes/MeshInfo2.cs
@@ -65,6 +65,8 @@ public MeshInfo2(SkinnedMeshRenderer renderer)
                 var bones = renderer.bones;
                 for (var i = 0; i < bones.Length && i < Bones.Count; i++) Bones[i].Transform = bones[i];
 
+                RemoveUnusedBones();
+
                 AssertInvariantContract("SkinnedMeshRenderer");
             });
         }
@@ -266,6 +268,11 @@ public void Clear()
         }
 
         public void Optimize()
+        {
+            RemoveUnusedBones();
+        }
+
+        private void RemoveUnusedBones()
         {
             // GC Bones
             var usedBones = new HashSet<Bone>();