Add defensive checks for min frame durations to be valid and non zero, fixing some MXF introspection bugs.

vade · vade · commit 6ddc63ced425 · 2024-11-07T15:02:44.000-05:00
diff --git a/Sources/OpenTimelineIO-AVFoundation/OpenTimelineIO-Extensions/Clip.swift b/Sources/OpenTimelineIO-AVFoundation/OpenTimelineIO-Extensions/Clip.swift
@@ -43,7 +43,9 @@ extension Clip
 
         var minFrameDuration:RationalTime? = nil
         if let videoTrack = asset.tracks(withMediaType: .video).first,
-            rescaleToAsset
+            rescaleToAsset,
+           videoTrack.minFrameDuration.isValid,
+           videoTrack.minFrameDuration != .zero
         {
             minFrameDuration = videoTrack.minFrameDuration.toOTIORationalTime()
         }

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,9 @@ extension Clip`
`43`	`43`
`44`	`44`	`var minFrameDuration:RationalTime? = nil`
`45`	`45`	`if let videoTrack = asset.tracks(withMediaType: .video).first,`
`46`		`- rescaleToAsset`
	`46`	`+ rescaleToAsset,`
	`47`	`+ videoTrack.minFrameDuration.isValid,`
	`48`	`+ videoTrack.minFrameDuration != .zero`
`47`	`49`	`{`
`48`	`50`	`minFrameDuration = videoTrack.minFrameDuration.toOTIORationalTime()`
`49`	`51`	`}`