Skip to content

Commit

Permalink
Merge pull request #80 from samulopez/fix/use-ConcurrentDictionary-fo…
Browse files Browse the repository at this point in the history
…r-translatedQuestNames

fix: use ConcurrentDictionary for translatedQuestNames
  • Loading branch information
lokinmodar authored Jul 11, 2024
2 parents dec192e + 2786be5 commit ae9e079
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion UiJournalHandler.cs
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@

using System;
using System.Collections.Generic;
using System.Collections.Concurrent;

using Dalamud.Game.Addon.Lifecycle;
using Dalamud.Game.Addon.Lifecycle.AddonArgTypes;
Expand All @@ -18,7 +19,7 @@ namespace Echoglossian
public partial class Echoglossian
{
// used to be sure we don't translate the same quest name twice
private readonly Dictionary<string, bool> translatedQuestNames = new();
private readonly ConcurrentDictionary<string, bool> translatedQuestNames = new();

private unsafe List<SummaryQuest> TranslateSummaries(
AtkComponentBase* journalBox,
Expand Down

0 comments on commit ae9e079

Please sign in to comment.