From: Jacob Casper <dev@jacobcasper.com>
Date: Fri, 17 Apr 2020 18:27:25 +0000 (-0500)
Subject: Defer isn't useful as we never leave the scope
X-Git-Url: https://git.jacobcasper.com/?p=brackets.git;a=commitdiff_plain;h=27d4a8e0b4d20fb92dfdd222e912b8026aed121d

Defer isn't useful as we never leave the scope
---

diff --git a/backend/scrape/graph/graph.go b/backend/scrape/graph/graph.go
index d043209..3198110 100644
--- 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
 			}