diff options
| author | Reto Brunner <reto@labrat.space> | 2021-01-30 11:33:31 +0100 |
|---|---|---|
| committer | Reto Brunner <reto@labrat.space> | 2021-01-30 14:04:23 +0100 |
| commit | 949781fa0a5f0654112b4f78558347ca991a89d3 (patch) | |
| tree | 1d8d58e66b567709ed746654ceb92d75667557b4 /commands/msg | |
| parent | 9385827cae7bab6534933718d21eeb489448c476 (diff) | |
| download | aerc-949781fa0a5f0654112b4f78558347ca991a89d3.tar.gz | |
Refactor lib/open to accept user provided arguments
* Get rid of open_darwin
It just lead to code duplication for a simple one string change.
Instead we query it during initialization
* Accept user provided arguments
"open" on MacOS accepts things like -A to use a specific application
Pass trough arguments the user provided in order to facilitate this
* Refactor the function to a struct
This makes it more convenient for the caller and avoids signatures like
lib.OpenFile(nil, u.String(), nil) which are fairly unreadable
Diffstat (limited to 'commands/msg')
| -rw-r--r-- | commands/msg/unsubscribe.go | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/commands/msg/unsubscribe.go b/commands/msg/unsubscribe.go index 205a255d..cf3e4a86 100644 --- a/commands/msg/unsubscribe.go +++ b/commands/msg/unsubscribe.go @@ -119,6 +119,5 @@ func unsubscribeMailto(aerc *widgets.Aerc, u *url.URL) error { } func unsubscribeHTTP(u *url.URL) error { - lib.OpenFile(u.String(), nil) - return nil + return lib.NewXDGOpen(u.String()).Start() } |