From b39a12aab79bc83f7edd84757ebac6d8d62c63b4 Mon Sep 17 00:00:00 2001
From: evalphobia <evalphobia@gmail.com>
Date: Thu, 5 Mar 2020 10:52:47 +0900
Subject: [PATCH] Handle panic error on client.Close() (#10)

---
 googlehome/client.go | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/googlehome/client.go b/googlehome/client.go
index cf17f89..dcd840a 100644
--- a/googlehome/client.go
+++ b/googlehome/client.go
@@ -83,7 +83,6 @@ func (c *Client) Notify(text string, language ...string) error {
 func (c *Client) Play(url string) (e error) {
 	client := cast.NewClient(c.ip, c.port)
 	defer func() {
-		client.Close()
 		if err := recover(); err != nil {
 			e = fmt.Errorf("Panic occurs on Play [%w]", err)
 		}
@@ -93,6 +92,7 @@ func (c *Client) Play(url string) (e error) {
 	if err != nil {
 		return err
 	}
+	defer client.Close()
 	client.Receiver().QuitApp(c.ctx)
 
 	media, err := client.Media(c.ctx)
@@ -113,7 +113,6 @@ func (c *Client) Play(url string) (e error) {
 func (c *Client) GetVolume() (volume float64, err error) {
 	client := cast.NewClient(c.ip, c.port)
 	defer func() {
-		client.Close()
 		if e := recover(); e != nil {
 			err = fmt.Errorf("Panic occurs on GetVolume [%w]", e)
 		}
@@ -123,6 +122,7 @@ func (c *Client) GetVolume() (volume float64, err error) {
 	if err != nil {
 		return 0, err
 	}
+	defer client.Close()
 
 	vol, err := client.Receiver().GetVolume(c.ctx)
 	if err != nil {
@@ -136,7 +136,6 @@ func (c *Client) GetVolume() (volume float64, err error) {
 func (c *Client) SetVolume(volume float64) (e error) {
 	client := cast.NewClient(c.ip, c.port)
 	defer func() {
-		client.Close()
 		if err := recover(); err != nil {
 			e = fmt.Errorf("Panic occurs on SetVolume [%w]", err)
 		}
@@ -146,6 +145,7 @@ func (c *Client) SetVolume(volume float64) (e error) {
 	if err != nil {
 		return err
 	}
+	defer client.Close()
 
 	_, err = client.Receiver().SetVolume(c.ctx, &controllers.Volume{Level: &volume})
 	return err
@@ -155,7 +155,6 @@ func (c *Client) SetVolume(volume float64) (e error) {
 func (c *Client) QuitApp() (e error) {
 	client := cast.NewClient(c.ip, c.port)
 	defer func() {
-		client.Close()
 		if err := recover(); err != nil {
 			e = fmt.Errorf("Panic occurs on QuitApp [%w]", err)
 		}
@@ -170,7 +169,6 @@ func (c *Client) QuitApp() (e error) {
 func (c *Client) StopMedia() (e error) {
 	client := cast.NewClient(c.ip, c.port)
 	defer func() {
-		client.Close()
 		if err := recover(); err != nil {
 			e = fmt.Errorf("Panic occurs on StopMedia [%w]", err)
 		}
@@ -190,13 +188,12 @@ func (c *Client) StopMedia() (e error) {
 
 func (c *Client) PauseMedia() error {
 	client := cast.NewClient(c.ip, c.port)
-	defer client.Close()
-	
+
 	media, err := client.Media(c.ctx)
 	if err != nil {
 		return err
 	}
-	
+
 	_, err = media.Pause(c.ctx)
-		return err
- }
+	return err
+}