*.go: go-staticcheck linting

Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>

keywords.go

@@ -174,9 +174,7 @@ func keyValDifference(this, that []KeyVal) []KeyVal {
 }
 func keyValCopy(set []KeyVal) []KeyVal {
 	ret := make([]KeyVal, len(set))
-	for i := range set {
+	copy(ret, set)
-		ret[i] = set[i]
-	}
 	return ret
 }
 

keywords_linux_test.go

@@ -4,7 +4,6 @@
 package mtree
 
 import (
-	"fmt"
 	"io/ioutil"
 	"os"
 	"path/filepath"
@@ -42,7 +41,7 @@ func TestXattr(t *testing.T) {
 	}
 
 	if err := xattr.Set(dir, "user.test", []byte("directory")); err != nil {
-		t.Skip(fmt.Sprintf("skipping: %q does not support xattrs", dir))
+		t.Skipf("skipping: %q does not support xattrs", dir)
 	}
 	if err := xattr.Set(filepath.Join(dir, "file"), "user.test", []byte("regular file")); err != nil {
 		t.Fatal(err)

tar.go

@@ -373,7 +373,6 @@ func flatten(root *Entry, creator *dhCreator, keywords []Keyword) {
 		}
 		creator.DH.Entries = append(creator.DH.Entries, dotEntry)
 	}
-	return
 }
 
 // resolveHardlinks goes through an Entry tree, and finds the Entry's associated
@@ -409,31 +408,6 @@ func resolveHardlinks(root *Entry, hardlinks map[string][]string, countlinks boo
 	}
 }
 
-// filter takes in a pointer to an Entry, and returns a slice of Entry's that
-// satisfy the predicate p
-func filter(root *Entry, p func(*Entry) bool) []Entry {
-	if root != nil {
-		var validEntrys []Entry
-		if len(root.Children) > 0 || root.Prev != nil {
-			for _, c := range root.Children {
-				// filter the sub-directory
-				if c.Prev != nil {
-					validEntrys = append(validEntrys, filter(c, p)...)
-				}
-				if p(c) {
-					if c.Prev == nil {
-						validEntrys = append([]Entry{*c}, validEntrys...)
-					} else {
-						validEntrys = append(validEntrys, *c)
-					}
-				}
-			}
-			return validEntrys
-		}
-	}
-	return nil
-}
-
 func (ts *tarStream) setErr(err error) {
 	ts.err = err
 }

tar_test.go

@@ -336,12 +336,6 @@ type fakeFile struct {
 	Xattrs     map[string]string
 }
 
-// minimal tar archive that mimics what is in ./testdata/test.tar
-var minimalFiles = []fakeFile{
-	{"x/", "", 0755, '5', 0, 0, nil},
-	{"x/files", "howdy\n", 0644, '0', 0, 0, nil},
-}
-
 func makeTarStream(ff []fakeFile) ([]byte, error) {
 	buf := new(bytes.Buffer)
 

update_linux_test.go

@@ -2,7 +2,6 @@ package mtree
 
 import (
 	"encoding/json"
-	"fmt"
 	"io/ioutil"
 	"os"
 	"path/filepath"
@@ -32,7 +31,7 @@ func TestXattrUpdate(t *testing.T) {
 	}
 
 	if err := xattr.Set(dir, "user.test", []byte("directory")); err != nil {
-		t.Skip(fmt.Sprintf("skipping: %q does not support xattrs", dir))
+		t.Skipf("skipping: %q does not support xattrs", dir)
 	}
 	if err := xattr.Set(tmpfn, "user.test", []byte("regular file")); err != nil {
 		t.Fatal(err)

walk.go

@@ -103,7 +103,7 @@ func Walk(root string, excludes []ExcludeFunc, keywords []Keyword, fsEval FsEval
 						}
 						keyFunc, ok := KeywordFuncs[keyword.Prefix()]
 						if !ok {
-							return fmt.Errorf("Unknown keyword %q for file %q", keyword.Prefix(), path)
+							return fmt.Errorf("unknown keyword %q for file %q", keyword.Prefix(), path)
 						}
 						kvs, err := creator.fs.KeywordFunc(keyFunc)(path, info, r)
 						if err != nil {
@@ -138,7 +138,7 @@ func Walk(root string, excludes []ExcludeFunc, keywords []Keyword, fsEval FsEval
 						}
 						keyFunc, ok := KeywordFuncs[keyword.Prefix()]
 						if !ok {
-							return fmt.Errorf("Unknown keyword %q for file %q", keyword.Prefix(), path)
+							return fmt.Errorf("unknown keyword %q for file %q", keyword.Prefix(), path)
 						}
 						kvs, err := creator.fs.KeywordFunc(keyFunc)(path, info, r)
 						if err != nil {
@@ -198,7 +198,7 @@ func Walk(root string, excludes []ExcludeFunc, keywords []Keyword, fsEval FsEval
 				}
 				keyFunc, ok := KeywordFuncs[keyword.Prefix()]
 				if !ok {
-					return fmt.Errorf("Unknown keyword %q for file %q", keyword.Prefix(), path)
+					return fmt.Errorf("unknown keyword %q for file %q", keyword.Prefix(), path)
 				}
 				kvs, err := creator.fs.KeywordFunc(keyFunc)(path, info, r)
 				if err != nil {