Merge pull request #59 from testwill/close

fix: check returned error before deferring file.Close()
jweny · Oct 30, 2023 · 269099d · 269099d
2 parents fa5ef34 + c6c415f
commit 269099d
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/file/file.go b/pkg/file/file.go
@@ -12,10 +12,10 @@ import (
 func ReadingLines(filename string) []string {
 	var result []string
 	file, err := os.Open(filename)
-	defer file.Close()
 	if err != nil {
 		return result
 	}
+	defer file.Close()
 
 	scanner := bufio.NewScanner(file)
 	for scanner.Scan() {