KurlesHS/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

bloblru: variable name cleanup

Browse source
This commit is contained in:
Michael Eischer 2024-05-26 12:38:41 +02:00
parent 21ce03cff2
commit 1c6067d93d
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
internal/bloblru/cache.go
View file

@ -97,13 +97,13 @@ func (c *Cache) GetOrCompute(id restic.ID, compute func() ([]byte, error)) ([]by
// check for parallel download or start our own // check for parallel download or start our own
finish := make(chan struct{}) finish := make(chan struct{})
c.mu.Lock() c.mu.Lock()
waitForResult, isDownloading := c.inProgress[id] waitForResult, isComputing := c.inProgress[id]
if !isDownloading { if !isComputing {
c.inProgress[id] = finish c.inProgress[id] = finish
} }
c.mu.Unlock() c.mu.Unlock()
if isDownloading { if isComputing {
// wait for result of parallel download // wait for result of parallel download
<-waitForResult <-waitForResult
} else { } 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, // 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. // thus the item might have been computed in the meantime.
// The following scenario would compute() the value multiple times otherwise: // The following scenario would compute() the value multiple times otherwise: