Collapse a few boolean operations

cmd/restic/integration_test.go

@@ -1417,11 +1417,7 @@ func linksEqual(source, dest map[uint64][]string) bool {
 		}
 	}
 
-	if len(dest) != 0 {
+	return len(dest) == 0
-		return false
-	}
-
-	return true
 }
 
 func linkEqual(source, dest []string) bool {

internal/archiver/archiver_test.go

@@ -1458,10 +1458,7 @@ func TestArchiverSnapshotSelect(t *testing.T) {
 				"other": TestFile{Content: "another file"},
 			},
 			selFn: func(item string, fi os.FileInfo) bool {
-				if filepath.Ext(item) == ".txt" {
+				return filepath.Ext(item) != ".txt"
-					return false
-				}
-				return true
 			},
 		},
 		{
@@ -1485,10 +1482,7 @@ func TestArchiverSnapshotSelect(t *testing.T) {
 				"other": TestFile{Content: "another file"},
 			},
 			selFn: func(item string, fi os.FileInfo) bool {
-				if filepath.Base(item) == "subdir" {
+				return filepath.Base(item) != "subdir"
-					return false
-				}
-				return true
 			},
 		},
 		{

internal/walker/walker.go

@@ -84,7 +84,7 @@ func walk(ctx context.Context, repo restic.TreeLoader, prefix string, parentTree
 				return false, err
 			}
 
-			if ignore == false {
+			if !ignore {
 				allNodesIgnored = false
 			}