ToxicMushroom/Dalamud
1
0
Fork 0
mirror of https://github.com/goatcorp/Dalamud.git synced 2025-12-12 18:27:23 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix: set lastPreferredRole in task to prevent race condition

Browse source
This commit is contained in:
goat 2019-12-08 16:28:54 +09:00
parent a91eeda956
commit 5730efa0b6
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Dalamud/Game/Network/NetworkHandlers.cs
View file

@ -65,9 +65,7 @@ namespace Dalamud.Game.Network {
return; return;
} }
Task.Run(async () => { Task.Run(async () => {
for (var rouletteIndex = 1; rouletteIndex < 11; rouletteIndex++) { for (var rouletteIndex = 1; rouletteIndex < 11; rouletteIndex++) {
var currentRole = data[16 + rouletteIndex]; var currentRole = data[16 + rouletteIndex];
var prevRole = this.lastPreferredRole[16 + rouletteIndex]; var prevRole = this.lastPreferredRole[16 + rouletteIndex];
@ -91,13 +89,15 @@ namespace Dalamud.Game.Network {
var prevRoleName = RoleKeyToName(prevRole); var prevRoleName = RoleKeyToName(prevRole);
var currentRoleName = RoleKeyToName(currentRole); var currentRoleName = RoleKeyToName(currentRole);
this.dalamud.Framework.Gui.Chat.Print($"Roulette bonus for {rouletteName} changed: {prevRole} => {currentRoleName}");
if (this.dalamud.BotManager.IsConnected) if (this.dalamud.BotManager.IsConnected)
await this.dalamud.BotManager.ProcessCfPreferredRoleChange(rouletteName, prevRoleName, currentRoleName); await this.dalamud.BotManager.ProcessCfPreferredRoleChange(rouletteName, prevRoleName, currentRoleName);
} }
} }
});
this.lastPreferredRole = data; this.lastPreferredRole = data;
});
return; return;
} }