From 0878b485ae0468142424d8bfa2b907effbfdc71b Mon Sep 17 00:00:00 2001
From: Motkov Kirill <motkov.kirill@gmail.com>
Date: Tue, 5 Mar 2019 23:10:04 +0300
Subject: [PATCH] remove suspicious err reassigns

---
 clean.go  |  2 +-
 config.go | 35 +++++++++++++++--------------------
 main.go   |  4 ++--
 3 files changed, 18 insertions(+), 23 deletions(-)

diff --git a/clean.go b/clean.go
index 4fc3dfb6..03393526 100644
--- a/clean.go
+++ b/clean.go
@@ -68,7 +68,7 @@ func syncClean(parser *arguments) error {
 	}
 
 	if mode == modeRepo || mode == modeAny {
-		if err = show(passToPacman(parser)); err != nil {
+		if err := show(passToPacman(parser)); err != nil {
 			return err
 		}
 	}
diff --git a/config.go b/config.go
index 9a4dcf25..8daebe26 100644
--- a/config.go
+++ b/config.go
@@ -352,7 +352,6 @@ func toUsage(usages []string) alpm.Usage {
 }
 
 func configureAlpm(conf *pacmanconf.Config) error {
-	var err error
 
 	// TODO: set SigLevel
 	//sigLevel := alpm.SigPackage | alpm.SigPackageOptional | alpm.SigDatabase | alpm.SigDatabaseOptional
@@ -371,68 +370,64 @@ func configureAlpm(conf *pacmanconf.Config) error {
 
 	}
 
-	if err = alpmHandle.SetCacheDirs(pacmanConf.CacheDir); err != nil {
+	if err := alpmHandle.SetCacheDirs(pacmanConf.CacheDir); err != nil {
 		return err
 	}
 
 	// add hook directories 1-by-1 to avoid overwriting the system directory
 	for _, dir := range pacmanConf.HookDir {
-		if err = alpmHandle.AddHookDir(dir); err != nil {
+		if err := alpmHandle.AddHookDir(dir); err != nil {
 			return err
 		}
 	}
 
-	if err = alpmHandle.SetGPGDir(pacmanConf.GPGDir); err != nil {
+	if err := alpmHandle.SetGPGDir(pacmanConf.GPGDir); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetLogFile(pacmanConf.LogFile); err != nil {
+	if err := alpmHandle.SetLogFile(pacmanConf.LogFile); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetIgnorePkgs(pacmanConf.IgnorePkg); err != nil {
+	if err := alpmHandle.SetIgnorePkgs(pacmanConf.IgnorePkg); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetIgnoreGroups(pacmanConf.IgnoreGroup); err != nil {
+	if err := alpmHandle.SetIgnoreGroups(pacmanConf.IgnoreGroup); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetArch(pacmanConf.Architecture); err != nil {
+	if err := alpmHandle.SetArch(pacmanConf.Architecture); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetNoUpgrades(pacmanConf.NoUpgrade); err != nil {
+	if err := alpmHandle.SetNoUpgrades(pacmanConf.NoUpgrade); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetNoExtracts(pacmanConf.NoExtract); err != nil {
+	if err := alpmHandle.SetNoExtracts(pacmanConf.NoExtract); err != nil {
 		return err
 	}
 
-	/*if err = alpmHandle.SetDefaultSigLevel(sigLevel); err != nil {
+	/*if err := alpmHandle.SetDefaultSigLevel(sigLevel); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetLocalFileSigLevel(localFileSigLevel); err != nil {
+	if err := alpmHandle.SetLocalFileSigLevel(localFileSigLevel); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetRemoteFileSigLevel(remoteFileSigLevel); err != nil {
+	if err := alpmHandle.SetRemoteFileSigLevel(remoteFileSigLevel); err != nil {
 		return err
 	}*/
 
-	if err = alpmHandle.SetDeltaRatio(pacmanConf.UseDelta); err != nil {
+	if err := alpmHandle.SetDeltaRatio(pacmanConf.UseDelta); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetUseSyslog(pacmanConf.UseSyslog); err != nil {
+	if err := alpmHandle.SetUseSyslog(pacmanConf.UseSyslog); err != nil {
 		return err
 	}
 
-	if err = alpmHandle.SetCheckSpace(pacmanConf.CheckSpace); err != nil {
-		return err
-	}
-
-	return nil
+	return alpmHandle.SetCheckSpace(pacmanConf.CheckSpace)
 }
diff --git a/main.go b/main.go
index 649e16b9..f3966d5a 100644
--- a/main.go
+++ b/main.go
@@ -142,7 +142,7 @@ func initAlpm() error {
 		pacmanConf.GPGDir = value
 	}
 
-	if err = initAlpmHandle(); err != nil {
+	if err := initAlpmHandle(); err != nil {
 		return err
 	}
 
@@ -173,7 +173,7 @@ func initAlpmHandle() error {
 		return fmt.Errorf("Unable to CreateHandle: %s", err)
 	}
 
-	if err = configureAlpm(pacmanConf); err != nil {
+	if err := configureAlpm(pacmanConf); err != nil {
 		return err
 	}