about summary refs log tree commit diff
path: root/tools/mpd-stats/cmd
diff options
context:
space:
mode:
authorFranck Cuny <franck@fcuny.net>2021-10-10 11:34:07 -0700
committerFranck Cuny <franck@fcuny.net>2022-06-11 14:32:07 -0700
commit14744fe82699d39b56d1acf795c7ffef38139b95 (patch)
tree8fada8cbbcc7b2eae807621eb143f1ca15574642 /tools/mpd-stats/cmd
parentmpd: rename function to create the player (diff)
downloadworld-14744fe82699d39b56d1acf795c7ffef38139b95.tar.gz
mpd: add function `Close` to the player
Let's close both the watcher and the client, instead of leaking this
interface to the user.
Diffstat (limited to 'tools/mpd-stats/cmd')
-rw-r--r--tools/mpd-stats/cmd/mpd-scrobbler/main.go7
1 files changed, 5 insertions, 2 deletions
diff --git a/tools/mpd-stats/cmd/mpd-scrobbler/main.go b/tools/mpd-stats/cmd/mpd-scrobbler/main.go
index 3953123..ba7bb05 100644
--- a/tools/mpd-stats/cmd/mpd-scrobbler/main.go
+++ b/tools/mpd-stats/cmd/mpd-scrobbler/main.go
@@ -16,8 +16,11 @@ func main() {
 		log.Fatalf("failed to create a client: %v", err)
 	}
 
-	defer c.Watcher.Close()
-	defer c.Client.Close()
+	defer func() {
+		if err := c.Close(); err != nil {
+			log.Fatalf("failed to close the player: %v", err)
+		}
+	}()
 
 	var (
 		currentRecord  *scrobbler.Record