no need to check for v nil

dahernan · dahernan · commit 5ab92b7b7639 · 2019-04-01T11:42:43.000+01:00
diff --git a/internal/mbhttp/client.go b/internal/mbhttp/client.go
@@ -56,10 +56,9 @@ func (c *Client) Do(req *http.Request, v interface{}) (*http.Response, error) {
 		}
 		return nil, errors.Errorf("%s: %s", c.boxname, o.Error)
 	}
-	if v == nil {
-		if err := json.Unmarshal(b, &v); err != nil {
-			return nil, errors.Wrap(err, "decode response data")
-		}
+	if err := json.Unmarshal(b, &v); err != nil {
+		return nil, errors.Wrap(err, "decode response data")
 	}
+
 	return resp, nil
 }

Original file line number	Diff line number	Diff line change
`@@ -56,10 +56,9 @@ func (c Client) Do(req http.Request, v interface{}) (*http.Response, error) {`
`56`	`56`	`}`
`57`	`57`	`return nil, errors.Errorf("%s: %s", c.boxname, o.Error)`
`58`	`58`	`}`
`59`		`- if v == nil {`
`60`		`- if err := json.Unmarshal(b, &v); err != nil {`
`61`		`- return nil, errors.Wrap(err, "decode response data")`
`62`		`- }`
	`59`	`+ if err := json.Unmarshal(b, &v); err != nil {`
	`60`	`+ return nil, errors.Wrap(err, "decode response data")`
`63`	`61`	`}`
	`62`	`+`
`64`	`63`	`return resp, nil`
`65`	`64`	`}`