Fix to close opened io resources as soon as not needed (#8839) (#8846)

* Fix to close opened io resources as soon as not needed

* Remove unneeded err checks
This commit is contained in:
Lauris BH 2019-11-05 22:14:56 +02:00 committed by GitHub
parent 26457782c1
commit 68b7f9f3f7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -252,6 +252,7 @@ func (g *GiteaLocalUploader) CreateReleases(releases ...*base.Release) error {
}
// download attachment
err = func() error {
resp, err := http.Get(asset.URL)
if err != nil {
return err
@ -269,10 +270,12 @@ func (g *GiteaLocalUploader) CreateReleases(releases ...*base.Release) error {
}
defer fw.Close()
if _, err := io.Copy(fw, resp.Body); err != nil {
_, err = io.Copy(fw, resp.Body)
return err
}()
if err != nil {
return err
}
rel.Attachments = append(rel.Attachments, &attach)
}
@ -468,21 +471,24 @@ func (g *GiteaLocalUploader) newPullRequest(pr *base.PullRequest) (*models.PullR
}
// download patch file
err := func() error {
resp, err := http.Get(pr.PatchURL)
if err != nil {
return nil, err
return err
}
defer resp.Body.Close()
pullDir := filepath.Join(g.repo.RepoPath(), "pulls")
if err = os.MkdirAll(pullDir, os.ModePerm); err != nil {
return nil, err
return err
}
f, err := os.Create(filepath.Join(pullDir, fmt.Sprintf("%d.patch", pr.Number)))
if err != nil {
return nil, err
return err
}
defer f.Close()
_, err = io.Copy(f, resp.Body)
return err
}()
if err != nil {
return nil, err
}
@ -496,8 +502,8 @@ func (g *GiteaLocalUploader) newPullRequest(pr *base.PullRequest) (*models.PullR
if err != nil {
return nil, err
}
defer p.Close()
_, err = p.WriteString(pr.Head.SHA)
p.Close()
if err != nil {
return nil, err
}
@ -531,8 +537,8 @@ func (g *GiteaLocalUploader) newPullRequest(pr *base.PullRequest) (*models.PullR
if err != nil {
return nil, err
}
defer b.Close()
_, err = b.WriteString(pr.Head.SHA)
b.Close()
if err != nil {
return nil, err
}