Skip to content

Commit d8b2b98

Browse files
committed
Remove a comment about an optimization that now exists in c/storage
Should not change behavior. Signed-off-by: Miloslav Trmač <mitr@redhat.com>
1 parent dff1269 commit d8b2b98

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

storage/storage_dest.go

-2
Original file line numberDiff line numberDiff line change
@@ -888,8 +888,6 @@ func (s *storageImageDestination) createNewLayer(index int, layerDigest digest.D
888888
// We can legitimately set storage.LayerOptions.OriginalDigest to "",
889889
// but that would just result in PutLayer computing the digest of the input stream == optionalDiffID.
890890
// So, instead, set .OriginalDigest to the value we know already, to avoid that digest computation.
891-
// FIXME? If both trustedUncompressedDigest and trustedOriginalDigest are "", PutLayer currently digests the uncompressed layer
892-
// twice. We could compute the digest here, but fixing that in c/storage would be more general.
893891
trustedOriginalDigest = optionalDiffID
894892

895893
// Allow using the already-collected layer contents without extracting the layer again.

0 commit comments

Comments
 (0)