Add a comment to clarify it's safe to close uatimezone

itingliu · itingliu · commit 1baa2eec84a7 · 2025-12-19T09:38:52.000+08:00
diff --git a/Sources/FoundationInternationalization/TimeZone/TimeZone_GMTICU.swift b/Sources/FoundationInternationalization/TimeZone/TimeZone_GMTICU.swift
@@ -91,6 +91,7 @@ internal final class _TimeZoneGMTICU : _TimeZoneProtocol, @unchecked Sendable {
             var status = U_ZERO_ERROR
             let tz = uatimezone_open($0.baseAddress, Int32($0.count), &status)
             defer {
+                // `uatimezone_close` checks for nil input, so it's safe to do it even there's an error.
                 uatimezone_close(tz)
             }
             guard status.isSuccess else {

Original file line number	Diff line number	Diff line change
`@@ -91,6 +91,7 @@ internal final class _TimeZoneGMTICU : _TimeZoneProtocol, @unchecked Sendable {`
`91`	`91`	`var status = U_ZERO_ERROR`
`92`	`92`	`let tz = uatimezone_open($0.baseAddress, Int32($0.count), &status)`
`93`	`93`	`defer {`
	`94`	+ // `uatimezone_close` checks for nil input, so it's safe to do it even there's an error.
`94`	`95`	`uatimezone_close(tz)`
`95`	`96`	`}`
`96`	`97`	`guard status.isSuccess else {`