simplify defer closers

Signed-off-by: Michael Sverdlov <[email protected]>

build/build.go

@@ -5,6 +5,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	ioutils "github.com/jfrog/gofrog/io"
 	"os"
 	"path/filepath"
 	"sort"
@@ -225,12 +226,7 @@ func (b *Build) SaveBuildInfo(buildInfo *entities.BuildInfo) (err error) {
 	if err != nil {
 		return
 	}
-	defer func() {
-		e := tempFile.Close()
-		if err == nil {
-			err = e
-		}
-	}()
+	defer ioutils.Close(tempFile, &err)
 	_, err = tempFile.Write(content.Bytes())
 	return
 }
@@ -257,12 +253,7 @@ func (b *Build) SavePartialBuildInfo(partial *entities.Partial) (err error) {
 	if err != nil {
 		return
 	}
-	defer func() {
-		e := tempFile.Close()
-		if err == nil {
-			err = e
-		}
-	}()
+	defer ioutils.Close(tempFile, &err)
 	_, err = tempFile.Write(content.Bytes())
 	return
 }

build/maven.go

@@ -173,10 +173,7 @@ func (mm *MavenModule) CalcDependencies() (err error) {
 	defer func() {
 		fileExist, e := utils.IsFileExists(mvnRunConfig.buildInfoProperties, false)
 		if fileExist && e == nil {
-			e = os.Remove(mvnRunConfig.buildInfoProperties)
-		}
-		if err == nil {
-			err = e
+			err = errors.Join(err, os.Remove(mvnRunConfig.buildInfoProperties))
 		}
 	}()
 	mvnRunConfig.SetOutputWriter(mm.outputWriter)

utils/checksum.go

@@ -2,18 +2,16 @@ package utils
 
 import (
 	"bufio"
-	"errors"
-
 	//#nosec G501 -- md5 is supported by Artifactory.
 	"crypto/md5"
 	//#nosec G505 -- sha1 is supported by Artifactory.
 	"crypto/sha1"
 	"fmt"
+	ioutils "github.com/jfrog/gofrog/io"
+	"github.com/minio/sha256-simd"
 	"hash"
 	"io"
 	"os"
-
-	"github.com/minio/sha256-simd"
 )
 
 type Algorithm int
@@ -37,9 +35,7 @@ func GetFileChecksums(filePath string, checksumType ...Algorithm) (checksums map
 	if err != nil {
 		return
 	}
-	defer func() {
-		err = errors.Join(err, file.Close())
-	}()
+	defer ioutils.Close(file, &err)
 	return CalcChecksums(file, checksumType...)
 }
 

utils/fileutils.go

@@ -5,6 +5,7 @@ import (
 	"encoding/json"
 	"errors"
 	"fmt"
+	ioutils "github.com/jfrog/gofrog/io"
 	"golang.org/x/exp/slices"
 	"io"
 	"net/http"
@@ -576,9 +577,7 @@ func GetFileDetails(filePath string, includeChecksums bool) (details *FileDetail
 	}
 
 	file, err := os.Open(filePath)
-	defer func() {
-		err = errors.Join(err, file.Close())
-	}()
+	defer ioutils.Close(file, &err)
 	if err != nil {
 		return
 	}
@@ -595,9 +594,7 @@ func calcChecksumDetails(filePath string) (checksum entities.Checksum, err error
 	if err != nil {
 		return
 	}
-	defer func() {
-		err = errors.Join(err, file.Close())
-	}()
+	defer ioutils.Close(file, &err)
 
 	checksums, err := CalcChecksums(file)
 	if err != nil {

utils/pythonutils/piputils.go

@@ -115,10 +115,7 @@ func getEgginfoPkginfoContent(setuppyFilePath string) (output []byte, err error)
 		return nil, err
 	}
 	defer func() {
-		e := utils.RemoveTempDir(eggBase)
-		if err == nil {
-			err = e
-		}
+		err = errors.Join(err, utils.RemoveTempDir(eggBase))
 	}()
 
 	// Run python 'egg_info --egg-base <eggBase>' command.