From 1c6067d93d61b33778d25f17da12481bdf7a5b84 Mon Sep 17 00:00:00 2001 From: Michael Eischer Date: Sun, 26 May 2024 12:38:41 +0200 Subject: [PATCH] bloblru: variable name cleanup --- internal/bloblru/cache.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/bloblru/cache.go b/internal/bloblru/cache.go index 161f15375..9981f8a87 100644 --- a/internal/bloblru/cache.go +++ b/internal/bloblru/cache.go @@ -97,13 +97,13 @@ func (c *Cache) GetOrCompute(id restic.ID, compute func() ([]byte, error)) ([]by // check for parallel download or start our own finish := make(chan struct{}) c.mu.Lock() - waitForResult, isDownloading := c.inProgress[id] - if !isDownloading { + waitForResult, isComputing := c.inProgress[id] + if !isComputing { c.inProgress[id] = finish } c.mu.Unlock() - if isDownloading { + if isComputing { // wait for result of parallel download <-waitForResult } else { @@ -116,7 +116,7 @@ func (c *Cache) GetOrCompute(id restic.ID, compute func() ([]byte, error)) ([]by }() } - // try again. This is necessary independent of whether isDownloading is true or not. + // try again. This is necessary independent of whether isComputing is true or not. // The calls to `c.Get()` and checking/adding the entry in `c.inProgress` are not atomic, // thus the item might have been computed in the meantime. // The following scenario would compute() the value multiple times otherwise: