TerribleDev/TimberWinR
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #21 from gcschorer/FixRedisOutputLeaks

Browse Source 
Single array remove instead of multiple
This commit is contained in:
Eric Fontana
2014-12-19 11:36:11 -05:00
parent 1063f493b5 8a69b58dfd
commit 27f3e114ef
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
TimberWinR/Outputs/Redis.cs
View File

@@ -145,10 +145,7 @@ namespace TimberWinR.Outputs
// then remove as many as necessary to get us under our limit
if (_queueOverflowDiscardOldest)
{
for (int i = 0; i <= (_jsonQueue.Count - _maxQueueSize); i++)
{
_jsonQueue.RemoveAt(0);
}
_jsonQueue.RemoveRange(0, (_jsonQueue.Count - _maxQueueSize) + 1);
}
// Otherwise we're in a "discard newest" mode, and this is the newest message, so just ignore it
else