I've also fixed a missing case for emote handling by making embedded content fallback to the HandleEdit function rather than the HandleMessage function as it triggers both a create and edit event.
I've also removed an unnecessary parameter from GetExistingBotReplyOrNull as it was not necessary.
Because of the nature of the application, messages are queued while UpdateAllEmotes is running, so DictEmotes.Clear() is a safe call and should never error when updating emotes UNLESS the array is searching at the same time as that is called. Possible fix may be required in the future.
Emotes are now in a dictionary!
I've also fixed a missing case for emote handling by making embedded content fallback to the
HandleEditfunction rather than theHandleMessagefunction as it triggers both a create and edit event.I've also removed an unnecessary parameter from
GetExistingBotReplyOrNullas it was not necessary.Because of the nature of the application, messages are queued while
UpdateAllEmotesis running, soDictEmotes.Clear()is a safe call and should never error when updating emotes UNLESS the array is searching at the same time as that is called. Possible fix may be required in the future.