Clean up code

Author: florianpircher

Sources/LightTableDelta/Delta.swift

@@ -124,11 +124,13 @@ extension Delta where Element: ~Copyable {
 		case .transition(let source, let target):
 			let source = try transform(source)
 			let target = try transform(target)
-			return if source != nil && target != nil {
-				.transition(source: source!, target: target!)
-			}
-			else if let source {
-				.source(source)
+			return if let source {
+				if let target {
+					.transition(source: source, target: target)
+				}
+				else {
+					.source(source)
+				}
 			}
 			else if let target {
 				.target(target)

Tests/LightTableDeltaTests/LightTableDeltaTests.swift

@@ -107,10 +107,10 @@ import LightTableDelta
 		(d1.endIndex ..< d1.endIndex, []),
 		(d1.startIndex ..< d1.endIndex, [3]),
 	]
-	for test in t1 {
-		#expect(d1[test.0].elementsEqual(test.1))
-		#expect(d1[test.0][test.0].elementsEqual(test.1))
-		#expect(d1[test.0][test.0][test.0].elementsEqual(test.1))
+	for (range, elements) in t1 {
+		#expect(d1[range].elementsEqual(elements))
+		#expect(d1[range][range].elementsEqual(elements))
+		#expect(d1[range][range][range].elementsEqual(elements))
 	}
 	#expect(d1[...].elementsEqual([3]))
 	#expect(d1[...][...].elementsEqual([3]))
@@ -122,10 +122,10 @@ import LightTableDelta
 		(d2.endIndex ..< d2.endIndex, []),
 		(d2.startIndex ..< d2.endIndex, [5]),
 	]
-	for test in t2 {
-		#expect(d2[test.0].elementsEqual(test.1))
-		#expect(d2[test.0][test.0].elementsEqual(test.1))
-		#expect(d2[test.0][test.0][test.0].elementsEqual(test.1))
+	for (range, elements) in t2 {
+		#expect(d2[range].elementsEqual(elements))
+		#expect(d2[range][range].elementsEqual(elements))
+		#expect(d2[range][range][range].elementsEqual(elements))
 	}
 	#expect(d2[...].elementsEqual([5]))
 	#expect(d2[...][...].elementsEqual([5]))
@@ -139,10 +139,10 @@ import LightTableDelta
 		(d3.index(after: d3.startIndex) ..< d3.endIndex, [5]),
 		(d3.startIndex ..< d3.endIndex, [3, 5]),
 	]
-	for test in t3 {
-		#expect(d3[test.0].elementsEqual(test.1))
-		#expect(d3[test.0][test.0].elementsEqual(test.1))
-		#expect(d3[test.0][test.0][test.0].elementsEqual(test.1))
+	for (range, elements) in t3 {
+		#expect(d3[range].elementsEqual(elements))
+		#expect(d3[range][range].elementsEqual(elements))
+		#expect(d3[range][range][range].elementsEqual(elements))
 	}
 	#expect(d3[...].elementsEqual([3, 5]))
 	#expect(d3[...][...].elementsEqual([3, 5]))