Disallow optional Text usage in builder body

davdroman · davdroman · commit 5057b669b80c · 2022-10-08T12:45:44.000+01:00
diff --git a/Sources/TextBuilder/TextBuilder.swift b/Sources/TextBuilder/TextBuilder.swift
@@ -61,7 +61,7 @@ public struct TextBuilderWith<Separator: TextBuilderSeparator> {
     }
 
     @inlinable
-    public static func buildExpression(_ component: Text?) -> Text? {
+    public static func buildExpression(_ component: Text) -> Text? {
         component
     }
 
@@ -109,12 +109,8 @@ public struct TextBuilderWith<Separator: TextBuilderSeparator> {
     }
 
     @inlinable
-    public static func buildExpression(_ text: Text?) -> [Text] {
-        if let text = text {
-            return [text]
-        } else {
-            return []
-        }
+    public static func buildExpression(_ text: Text) -> [Text] {
+        [text]
     }
 
     @inlinable

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ public struct TextBuilderWith<Separator: TextBuilderSeparator> {`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`@inlinable`
`64`		`- public static func buildExpression(_ component: Text?) -> Text? {`
	`64`	`+ public static func buildExpression(_ component: Text) -> Text? {`
`65`	`65`	`component`
`66`	`66`	`}`
`67`	`67`
`@@ -109,12 +109,8 @@ public struct TextBuilderWith<Separator: TextBuilderSeparator> {`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`@inlinable`
`112`		`- public static func buildExpression(_ text: Text?) -> [Text] {`
`113`		`- if let text = text {`
`114`		`- return [text]`
`115`		`- } else {`
`116`		`- return []`
`117`		`- }`
	`112`	`+ public static func buildExpression(_ text: Text) -> [Text] {`
	`113`	`+ [text]`
`118`	`114`	`}`
`119`	`115`
`120`	`116`	`@inlinable`