passepartoutvpn
/
wireguard-apple
mirror of https://github.com/passepartoutvpn/wireguard-apple.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

TunnelsManager: startDeactivation() need not take a completion handler 

Because the completion handler pattern doesn't fit in this case.

Signed-off-by: Roopesh Chander <roop@roopc.net>
This commit is contained in:
Roopesh Chander 2018-11-10 12:25:17 +05:30
parent 95456ec956
commit 8e7bfb15ed
3 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
WireGuard/WireGuard/UI/iOS/TunnelDetailTableViewController.swift
View File

@ -159,9 +159,7 @@ extension TunnelDetailTableViewController {
}
}
} else {
s.tunnelsManager.startDeactivation(of: s.tunnel) { error in
print("Error while deactivating: \(String(describing: error))")
}
s.tunnelsManager.startDeactivation(of: s.tunnel)
}
}
return cell

4
WireGuard/WireGuard/UI/iOS/TunnelsListTableViewController.swift
View File

@ -287,9 +287,7 @@ extension TunnelsListTableViewController: UITableViewDataSource {
}
}
} else {
tunnelsManager.startDeactivation(of: tunnel) { [weak s] error in
s?.showErrorAlert(title: "Deactivation error", message: "Error while bringing down tunnel: \(String(describing: error))")
}
tunnelsManager.startDeactivation(of: tunnel)
}
}
}

3
WireGuard/WireGuard/VPN/TunnelsManager.swift
View File

@ -200,9 +200,8 @@ class TunnelsManager {
tunnel.startActivation(completionHandler: completionHandler)
}
func startDeactivation(of tunnel: TunnelContainer, completionHandler: @escaping (Error?) -> Void) {
func startDeactivation(of tunnel: TunnelContainer) {
if (tunnel.status == .inactive) {
completionHandler(TunnelActivationError.attemptingDeactivationWhenTunnelIsInactive)
return
}
tunnel.startDeactivation()