Defer isn't useful as we never leave the scope

[brackets.git] / backend / scrape / graph / graph.go

diff --git a/backend/scrape/graph/graph.go b/backend/scrape/graph/graph.go
index cc1dc1e..3198110 100644 (file)
--- a/backend/scrape/graph/graph.go
+++ b/backend/scrape/graph/graph.go
@@ -26,18 +26,19 @@ WHERE ID NOT IN (
                        log.Print(err)
                        continue infinite
                }
-               defer rows.Close()
 
                var artistId string
                for rows.Next() {
                        if err := rows.Scan(&artistId); err != nil {
                                log.Print(err)
+                               rows.Close()
                                continue infinite
                        }
 
                        artists, err := env.C.GetRelatedArtists(spotify.ID(artistId))
                        if err != nil {
                                log.Print(err)
+                               rows.Close()
                                continue infinite
                        }
 
@@ -62,6 +63,7 @@ SELECT EXISTS (
                                }
 
                                resp, err := http.PostForm("http://localhost:8080/artist/add", url.Values{"id": {string(artist.ID)}})
+                               resp.Body.Close()
                                if err != nil {
                                        log.Print(err)
                                        success = false