Skip to content

Conversation

@Haselnussbomber
Copy link
Member

There isn't really a need to check for it anyways and it just blows up when plugins try to access something that is guarded by this during disposal when the game is shutting down.

@Haselnussbomber Haselnussbomber requested a review from a team as a code owner October 28, 2025 00:11
@Haselnussbomber
Copy link
Member Author

goat in #dalamud-dev:

That isn't actually thread-safe

So I'm closing this.

Plugins are better off checking Framework.IsFrameworkUnloading in their Dispose function.

@Haselnussbomber Haselnussbomber deleted the no-threadsafety-when-shutting-down branch November 11, 2025 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant