From da8b168573ee0771f4a661a335f09f0c2541c40a Mon Sep 17 00:00:00 2001
From: Christian Schwarz <me@cschwarz.com>
Date: Sun, 5 Apr 2020 20:57:49 +0200
Subject: [PATCH] zfs changes fixup

---
 platformtest/tests/helpers.go |  4 ++--
 zfs/versions.go               |  2 +-
 zfs/zfs.go                    | 19 ++++++++-----------
 3 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/platformtest/tests/helpers.go b/platformtest/tests/helpers.go
index 72c940a..bb7f516 100644
--- a/platformtest/tests/helpers.go
+++ b/platformtest/tests/helpers.go
@@ -157,7 +157,7 @@ func versionRelnamesSorted(versions []zfs.FilesystemVersion) []string {
 	for _, v := range versions {
 		vstrs = append(vstrs, v.RelName())
 	}
-	sort.Sort(sort.StringSlice(vstrs))
+	sort.Strings(vstrs)
 	return vstrs
 }
 
@@ -171,6 +171,6 @@ func datasetToStringSortedTrimPrefix(prefix *zfs.DatasetPath, paths []*zfs.Datas
 		}
 		pstrs = append(pstrs, trimmed.ToString())
 	}
-	sort.Sort(sort.StringSlice(pstrs))
+	sort.Strings(pstrs)
 	return pstrs
 }
diff --git a/zfs/versions.go b/zfs/versions.go
index 88c9935..fed3d60 100644
--- a/zfs/versions.go
+++ b/zfs/versions.go
@@ -37,7 +37,7 @@ var (
 
 func (s VersionTypeSet) zfsListTFlagRepr() string {
 	var types []string
-	for t, _ := range s {
+	for t := range s {
 		types = append(types, t.String())
 	}
 	return strings.Join(types, ",")
diff --git a/zfs/zfs.go b/zfs/zfs.go
index 8e7c842..1d61f6c 100644
--- a/zfs/zfs.go
+++ b/zfs/zfs.go
@@ -1086,18 +1086,15 @@ func ZFSRecv(ctx context.Context, fs string, v *ZFSSendArgVersion, streamCopier
 	if opts.RollbackAndForceRecv {
 		// destroy all snapshots before `recv -F` because `recv -F`
 		// does not perform a rollback unless `send -R` was used (which we assume hasn't been the case)
-		var snaps []FilesystemVersion
-		{
-			snaps, err := ZFSListFilesystemVersions(fsdp, ListFilesystemVersionsOptions{
-				Types: Snapshots,
-			})
-			if err != nil {
-				return fmt.Errorf("cannot list versions for rollback for forced receive: %s", err)
-			}
-			sort.Slice(snaps, func(i, j int) bool {
-				return snaps[i].CreateTXG < snaps[j].CreateTXG
-			})
+		snaps, err := ZFSListFilesystemVersions(fsdp, ListFilesystemVersionsOptions{
+			Types: Snapshots,
+		})
+		if err != nil {
+			return fmt.Errorf("cannot list versions for rollback for forced receive: %s", err)
 		}
+		sort.Slice(snaps, func(i, j int) bool {
+			return snaps[i].CreateTXG < snaps[j].CreateTXG
+		})
 		// bookmarks are rolled back automatically
 		if len(snaps) > 0 {
 			// use rollback to efficiently destroy all but the earliest snapshot