Просмотр исходного кода

tech(cli): dont use defer statements in loops

bergquist 9 лет назад
Родитель
Сommit
903d1b7797
1 измененных файлов с 2 добавлено и 2 удалено
  1. 2 2
      pkg/cmd/grafana-cli/commands/install_command.go

+ 2 - 2
pkg/cmd/grafana-cli/commands/install_command.go

@@ -164,14 +164,14 @@ func downloadFile(pluginName, filePath, url string) (err error) {
 				return fmt.Errorf(permissionsDeniedMessage, newFile)
 			}
 
-			defer dst.Close()
 			src, err := zf.Open()
 			if err != nil {
 				log.Errorf("%v", err)
 			}
-			defer src.Close()
 
 			io.Copy(dst, src)
+			dst.Close()
+			src.Close()
 		}
 	}