builtin: deprecate FirstNonEmpty() in favor of stdlib's cmp.Or()

joonas-fi · joonas-fi · commit c43746e4ead9 · 2025-06-04T09:29:04.000+03:00
diff --git a/builtin/builtin.go b/builtin/builtin.go
@@ -55,6 +55,7 @@ func ErrorIfUnset(isUnset bool, fieldName string) error {
 }
 
 // returns the first non-empty value.
+// Deprecated: use `cmp.Or()`
 func FirstNonEmpty[T comparable](values ...T) T {
 	var empty T
 	for _, value := range values {

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,7 @@ func ErrorIfUnset(isUnset bool, fieldName string) error {`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`// returns the first non-empty value.`
	`58`	+// Deprecated: use `cmp.Or()`
`58`	`59`	`func FirstNonEmpty[T comparable](values ...T) T {`
`59`	`60`	`var empty T`
`60`	`61`	`for _, value := range values {`