Fix availability for NWEndpoint (#51)
Motivation:
`NWEndpoint` added a `.url` case for macOS 10.15 which commit afbbead added
support for. However, in doing so it also broke support for 10.14.
Modifications:
Handle `.url` in the `default` case and also remove `@unknown` from the
cases of the switch statements in NIOTSListenerChannel and SocketAddress
init.
Result:
Compiles on 10.14.
This commit is contained in:
parent
23e77df42a
commit
eec3aed641
|
|
@ -301,9 +301,10 @@ extension NIOTSListenerChannel: StateManagedChannel {
|
||||||
parameters.requiredLocalEndpoint = target
|
parameters.requiredLocalEndpoint = target
|
||||||
case .service(_, _, _, let interface):
|
case .service(_, _, _, let interface):
|
||||||
parameters.requiredInterface = interface
|
parameters.requiredInterface = interface
|
||||||
case .url:
|
default:
|
||||||
break
|
// We can't use `@unknown default` and explicitly list cases we know about since they
|
||||||
@unknown default:
|
// would require availability checks within the switch statement (`.url` was added in
|
||||||
|
// macOS 10.15).
|
||||||
()
|
()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -100,9 +100,12 @@ extension SocketAddress {
|
||||||
self = .init(addr, host: host.debugDescription)
|
self = .init(addr, host: host.debugDescription)
|
||||||
case .unix(let path):
|
case .unix(let path):
|
||||||
self = try .init(unixDomainSocketPath: path)
|
self = try .init(unixDomainSocketPath: path)
|
||||||
case .service, .hostPort, .url:
|
case .service, .hostPort:
|
||||||
throw NIOTSErrors.UnableToResolveEndpoint()
|
throw NIOTSErrors.UnableToResolveEndpoint()
|
||||||
@unknown default:
|
default:
|
||||||
|
// We can't use `@unknown default` and explicitly list cases we know about since they
|
||||||
|
// would require availability checks within the switch statement (`.url` was added in
|
||||||
|
// macOS 10.15).
|
||||||
throw NIOTSErrors.UnableToResolveEndpoint()
|
throw NIOTSErrors.UnableToResolveEndpoint()
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue