prehistoric-systems/Flow.Launcher
1
0
Fork 0
mirror of https://github.com/Flow-Launcher/Flow.Launcher.git synced 2026-03-11 08:54:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fix TryTake may remove the wrong element

Browse source
This commit is contained in:
Jack251970 2025-05-01 13:42:54 +08:00
parent 4ecef470e8
commit c49b3b7cba
1 changed files with 5 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
Flow.Launcher/Storage/TopMostRecord.cs
View file

@ -189,11 +189,8 @@ namespace Flow.Launcher.Storage
}
// remove the record from the bag
var recordToRemove = value.FirstOrDefault(r => r.Equals(result));
if (recordToRemove != null)
{
value.TryTake(out recordToRemove);
}
var bag = new ConcurrentQueue<Record>(value.Where(r => !r.Equals(result)));
records[result.OriginQuery.RawQuery] = new ConcurrentBag<Record>(bag);
// if the bag is empty, remove the bag from the dictionary
if (value.IsEmpty)
@ -230,21 +227,9 @@ namespace Flow.Launcher.Storage
else
{
// add or update the record in the bag
if (value.Any(r => r.Equals(result)))
{
// update the record
var recordToUpdate = value.FirstOrDefault(r => r.Equals(result));
if (recordToUpdate != null)
{
value.TryTake(out recordToUpdate);
value.Add(record);
}
}
else
{
// add the record
value.Add(record);
}
var bag = new ConcurrentQueue<Record>(value.Where(r => !r.Equals(result))); // make sure we don't have duplicates
bag.Enqueue(record);
records[result.OriginQuery.RawQuery] = new ConcurrentBag<Record>(bag);
}
}
}