[cache] Use slice.ContainsFunc

cleaner than a basic for loop

Signed-off-by: Baptiste Girard-Carrabin <[email protected]>

Author: Fricounet

cache/remote.go

@@ -119,12 +119,11 @@ func getAvailableBlobs(ctx context.Context, cs content.Store, chain *solver.Remo
 		// Looping over the list is preferable:
 		// 1. to avoid using a map, which don't preserve the order of descriptors,
 		// 2. descs will have a length the number of compression variants for a blob, which is usually very small
-		for _, d := range descs {
-			if d.Digest == desc.Digest {
-				return true // already seen this descriptor
-			}
+		if !slices.ContainsFunc(descs, func(d ocispecs.Descriptor) bool {
+			return d.Digest == desc.Digest
+		}) {
+			descs = append(descs, desc)
 		}
-		descs = append(descs, desc)
 		return true
 	}); err != nil {
 		bklog.G(ctx).WithError(err).Warn("failed to walk variant blob") // is not a critical error at this moment.