spdf/elektron
Archived
1
0
Fork 0
Code Issues 4 Pull requests 1 Projects Releases Packages Wiki Activity

fixed bug in Remove() where the currentSum and numberOfTasksInWindow weren't getting updated.

Browse source
This commit is contained in:
Pradyumna Kaushik 2016-12-20 13:53:28 -05:00 • committed by Renan DelValle
parent e0a16da97a
commit 1b821735ed
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
utilities/runAvg/runAvg.go
View file

@ -74,6 +74,8 @@ func (rac *runningAverageCalculator) removeFromWindow(id string) (interface{}, e
for element := rac.window.Front(); element != nil; element = element.Next() { for element := rac.window.Front(); element != nil; element = element.Next() {
if elementToRemove := element.Value.(Interface); elementToRemove.ID() == id { if elementToRemove := element.Value.(Interface); elementToRemove.ID() == id {
rac.window.Remove(element) rac.window.Remove(element)
rac.currentSum -= elementToRemove.Val()
rac.numberOfElementsInWindow--
return elementToRemove, nil return elementToRemove, nil
} }
} }