Throw an error if a URL's reported path is not absolute

owenv · owenv · commit 2d3407e0da64 · 2025-05-19T17:55:51.000-07:00
diff --git a/Sources/SWBUtil/URL.swift b/Sources/SWBUtil/URL.swift
@@ -28,7 +28,9 @@ extension URL {
                     throw FileURLError.notRepresentable(self)
                 }
                 let fp = Path(String(cString: cString))
-                precondition(fp.isAbsolute, "path '\(fp.str)' is not absolute")
+                guard fp.isAbsolute else {
+                    throw FileURLError.pathNotAbsolute(self)
+                }
                 return fp
             }
         }
@@ -37,4 +39,5 @@ extension URL {
 
 fileprivate enum FileURLError: Error {
     case notRepresentable(URL)
+    case pathNotAbsolute(URL)
 }

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,9 @@ extension URL {`
`28`	`28`	`throw FileURLError.notRepresentable(self)`
`29`	`29`	`}`
`30`	`30`	`let fp = Path(String(cString: cString))`
`31`		`- precondition(fp.isAbsolute, "path '\(fp.str)' is not absolute")`
	`31`	`+ guard fp.isAbsolute else {`
	`32`	`+ throw FileURLError.pathNotAbsolute(self)`
	`33`	`+ }`
`32`	`34`	`return fp`
`33`	`35`	`}`
`34`	`36`	`}`
`@@ -37,4 +39,5 @@ extension URL {`
`37`	`39`
`38`	`40`	`fileprivate enum FileURLError: Error {`
`39`	`41`	`case notRepresentable(URL)`
	`42`	`+ case pathNotAbsolute(URL)`
`40`	`43`	`}`