Skip to content

fix #302 handycontrol language set causing deadlock#303

Merged
DineshSolanki merged 1 commit into
masterfrom
302-published-app-startup-deadlock-caused-by-eager-initialization-of-handycontrol-confighelper
Jun 23, 2026
Merged

fix #302 handycontrol language set causing deadlock#303
DineshSolanki merged 1 commit into
masterfrom
302-published-app-startup-deadlock-caused-by-eager-initialization-of-handycontrol-confighelper

Conversation

@DineshSolanki

Copy link
Copy Markdown
Owner

The call to ConfigHelper.Instance inside CultureUtils.GetCultureInfoByLanguage() was wrapped inside a deferred dispatcher block using Application.Current.Dispatcher.BeginInvoke with a priority of DispatcherPriority.Background. This allows the method to instantly return the CultureInfo to the constructor, freeing the application lifecycle to render the shell window before the HandyControl theme engine is initialized.

The call to `ConfigHelper.Instance` inside `CultureUtils.GetCultureInfoByLanguage()` was wrapped inside a deferred dispatcher block using `Application.Current.Dispatcher.BeginInvoke` with a priority of `DispatcherPriority.Background`. This allows the method to instantly return the `CultureInfo` to the constructor, freeing the application lifecycle to render the shell window before the HandyControl theme engine is initialized.
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@sonarqubecloud

Copy link
Copy Markdown

@DineshSolanki DineshSolanki merged commit fcbacc3 into master Jun 23, 2026
10 checks passed
@DineshSolanki DineshSolanki deleted the 302-published-app-startup-deadlock-caused-by-eager-initialization-of-handycontrol-confighelper branch June 23, 2026 20:25
DineshSolanki added a commit that referenced this pull request Jun 24, 2026
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.

Published app startup deadlock, caused by eager initialization of HandyControl ConfigHelper

1 participant