From 5ca73b911fb3769e1376a15a8da1cee22f1207d3 Mon Sep 17 00:00:00 2001 From: Jakub Sztandera Date: Wed, 24 Jun 2020 17:23:42 +0200 Subject: [PATCH] Use append instead of index assign Co-authored-by: Steven Allen --- blockservice.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/blockservice.go b/blockservice.go index 9cf96a8..33f6914 100644 --- a/blockservice.go +++ b/blockservice.go @@ -282,18 +282,16 @@ func getBlocks(ctx context.Context, ks []cid.Cid, bs blockstore.Blockstore, fget } if !allValid { - ks2 := make([]cid.Cid, len(ks)) - k := 0 + ks2 := make([]cid.Cid, 0, len(ks)) for _, c := range ks { // hash security if err := verifcid.ValidateCid(c); err == nil { - ks2[k] = c - k++ + ks2 = append(ks2, c) } else { log.Errorf("unsafe CID (%s) passed to blockService.GetBlocks: %s", c, err) } } - ks = ks2[:k] + ks = ks2 } var misses []cid.Cid -- GitLab