refactor: remove redundant len check (#2291)

`len` returns 0 if the slice or map is nil. From the Go specification
[1]:

  "1. For a nil slice, the number of iterations is 0."
  "3. If the map is nil, the number of iterations is 0."

Therefore, an additional `len(v) != 0` check for before the loop is
unnecessary.

[1]: https://go.dev/ref/spec#For_range

Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
This commit is contained in:
Eng Zer Jun 2023-09-27 18:36:08 +08:00 committed by GitHub
parent 299aa1e123
commit a1121556be
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 8 deletions

2
get.go
View File

@ -26,11 +26,9 @@ func printPkgbuilds(dbExecutor download.DBSearcher, aurClient aur.QueryClient,
logger.Errorln(err)
}
if len(pkgbuilds) != 0 {
for target, pkgbuild := range pkgbuilds {
logger.Printf("\n\n# %s\n\n%s", target, string(pkgbuild))
}
}
if len(pkgbuilds) != len(targets) {
missing := []string{}

View File

@ -80,11 +80,9 @@ func syncInfo(ctx context.Context, run *runtime.Runtime,
missing = true
}
if len(info) != 0 {
for i := range info {
printInfo(run.Logger, run.Cfg, &info[i], cmdArgs.ExistsDouble("i"))
}
}
if missing {
err = fmt.Errorf("")