Don't crash if trimming fails for some codecs (#257)

Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>

Gifski/Utilities.swift

@@ -604,6 +604,7 @@ extension AVAssetTrack {
 		case videoTrackIsEmpty
 		case assetIsMissingVideoTrack
 		case compositionCouldNotBeCreated
+		case codecNotSupported
 	}
 
 	/**
@@ -612,6 +613,16 @@ extension AVAssetTrack {
 	This can be useful to trim blank frames from files produced by tools like the iOS simulator screen recorder.
 	*/
 	func trimmingBlankFrames() throws -> AVAssetTrack {
+		// See https://github.com/sindresorhus/Gifski/issues/254 for context.
+		// In short: Some codecs seem to always report a sample buffer size of 0 when reading, breaking this function. (macOS 11.6)
+		let buggyCodecs = ["v210", "BGRA"]
+		if
+			let codecIdentifier = codecIdentifier,
+			buggyCodecs.contains(codecIdentifier)
+		{
+			throw VideoTrimmingError.codecNotSupported
+		}
+
 		// Create new composition
 		let composition = AVMutableComposition()
 		guard
@@ -665,6 +676,8 @@ extension AVAssetTrack.VideoTrimmingError: LocalizedError {
 			return "Asset is missing video track."
 		case .compositionCouldNotBeCreated:
 			return "Composition could not be created."
+		case .codecNotSupported:
+			return "Video codec is not supported."
 		}
 	}
 }

Gifski/VideoValidator.swift

@@ -188,6 +188,9 @@ struct VideoValidator {
 		do {
 			let trimmedAsset = try newAsset.trimmingBlankFramesFromFirstVideoTrack()
 			return .success(trimmedAsset, newVideoMetadata)
+		} catch AVAssetTrack.VideoTrimmingError.codecNotSupported {
+			// Allow user to continue
+			return .success(newAsset, newVideoMetadata)
 		} catch {
 			NSAlert.showModalAndReportToCrashlytics(
 				for: window,