add collect dependencies that takes a single anim clip

Author: vkovec

Packages/com.unity.formats.fbx/Editor/Scripts/FbxExporter.cs

@@ -2944,15 +2944,12 @@ public static IExportData GetExportData(GameObject rootObject, AnimationClip ani
                     exportOptions = DefaultOptions;
                 Debug.Assert(exportOptions!=null);
 
-                // get animation clips for root object from animation track
-                List<AnimationClip> clips = new List<AnimationClip>(){animationClip};
-
                 var goToExport = new HashSet<GameObject>();
                 var animationClips = new Dictionary<AnimationClip, GameObject>();
                 var exportComponent = new Dictionary<GameObject, System.Type>();
 
                 var exportData = new AnimationOnlyExportData(animationClips, goToExport, exportComponent);
-                exportData.CollectDependencies(clips.ToArray(), rootObject, exportOptions);
+                exportData.CollectDependencies(animationClip, rootObject, exportOptions);
 
                 return exportData;
             }

Packages/com.unity.formats.fbx/Editor/Scripts/IExportData.cs

@@ -47,57 +47,72 @@ public AnimationOnlyExportData(
         }
 
         /// <summary>
-        /// collect all objects dependencies for animation clips.
+        /// collect all object dependencies for given animation clip
         /// </summary>
         public void CollectDependencies(
-            AnimationClip[] animClips,
+            AnimationClip animClip,
             GameObject rootObject,
             IExportOptions exportOptions
         )
         {
             Debug.Assert(rootObject != null);
             Debug.Assert(exportOptions != null);
 
+            if (this.animationClips.ContainsKey(animClip))
+            {
+                // we have already exported gameobjects for this clip
+                return;
+            }
+
             // NOTE: the object (animationRootObject) containing the animation is not necessarily animated
             // when driven by an animator or animation component.
-            foreach (var animClip in animClips)
+            this.animationClips.Add(animClip, rootObject);
+
+            foreach (EditorCurveBinding uniCurveBinding in AnimationUtility.GetCurveBindings(animClip))
             {
-                if (this.animationClips.ContainsKey(animClip))
+                Object uniObj = AnimationUtility.GetAnimatedObject(rootObject, uniCurveBinding);
+                if (!uniObj)
                 {
-                    // we have already exported gameobjects for this clip
-                    continue;
+                    return;
                 }
 
-                this.animationClips.Add(animClip, rootObject);
+                GameObject unityGo = ModelExporter.GetGameObject(uniObj);
+                if (!unityGo)
+                {
+                    return;
+                }
 
-                foreach (EditorCurveBinding uniCurveBinding in AnimationUtility.GetCurveBindings(animClip))
+                if (!exportOptions.AnimateSkinnedMesh && unityGo.GetComponent<SkinnedMeshRenderer>())
                 {
-                    Object uniObj = AnimationUtility.GetAnimatedObject(rootObject, uniCurveBinding);
-                    if (!uniObj)
-                    {
-                        continue;
-                    }
-
-                    GameObject unityGo = ModelExporter.GetGameObject(uniObj);
-                    if (!unityGo)
-                    {
-                        continue;
-                    }
-
-                    if (!exportOptions.AnimateSkinnedMesh && unityGo.GetComponent<SkinnedMeshRenderer>())
-                    {
-                        continue;
-                    }
-
-                    // If we have a clip driving a camera or light then force the export of FbxNodeAttribute
-                    // so that they point the right way when imported into Maya.
-                    if (unityGo.GetComponent<Light>())
-                        this.exportComponent[unityGo] = typeof(Light);
-                    else if (unityGo.GetComponent<Camera>())
-                        this.exportComponent[unityGo] = typeof(Camera);
-
-                    this.goExportSet.Add(unityGo);
+                    return;
                 }
+
+                // If we have a clip driving a camera or light then force the export of FbxNodeAttribute
+                // so that they point the right way when imported into Maya.
+                if (unityGo.GetComponent<Light>())
+                    this.exportComponent[unityGo] = typeof(Light);
+                else if (unityGo.GetComponent<Camera>())
+                    this.exportComponent[unityGo] = typeof(Camera);
+
+                this.goExportSet.Add(unityGo);
+            }
+        }
+
+        /// <summary>
+        /// collect all objects dependencies for animation clips.
+        /// </summary>
+        public void CollectDependencies(
+            AnimationClip[] animClips,
+            GameObject rootObject,
+            IExportOptions exportOptions
+        )
+        {
+            Debug.Assert(rootObject != null);
+            Debug.Assert(exportOptions != null);
+            
+            foreach (var animClip in animClips)
+            {
+                CollectDependencies(animClip, rootObject, exportOptions);
             }
         }