From f8c4dd7b1a938067152443ef5f1884f1b40c8cb1 Mon Sep 17 00:00:00 2001 From: Alexander Neumann Date: Tue, 3 Nov 2020 12:50:47 +0100 Subject: [PATCH] Split packe rewrite logic into two case branches The comma is too sublte, let's split this into two separate branches. --- cmd/restic/cmd_prune.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/cmd/restic/cmd_prune.go b/cmd/restic/cmd_prune.go index 90b702e61..4f83a9874 100644 --- a/cmd/restic/cmd_prune.go +++ b/cmd/restic/cmd_prune.go @@ -315,9 +315,11 @@ func prune(opts PruneOptions, gopts GlobalOptions, repo restic.Repository, usedB stats.blobs.remove += p.unusedBlobs stats.size.remove += p.unusedSize - case opts.RepackCachableOnly && p.tpe == restic.DataBlob, + case opts.RepackCachableOnly && p.tpe == restic.DataBlob: // if this is a data pack and --repack-cacheable-only is set => keep pack! - p.unusedBlobs == 0 && p.duplicateBlobs == 0 && p.tpe != restic.InvalidBlob: + stats.packs.keep++ + + case p.unusedBlobs == 0 && p.duplicateBlobs == 0 && p.tpe != restic.InvalidBlob: // All blobs in pack are used and not duplicates/mixed => keep pack! stats.packs.keep++