Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Perform the persistence asynchronously to not block the main (UI) thread #69

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

adangel
Copy link
Member

@adangel adangel commented Jun 9, 2023

This avoids "your app is not responding" errors, while the git command is waiting for password input to load the gpg key for commit signing.

This also reverts the fix from 7cc0463: Calling System#exit immediately after Platform#exit prevents Application#stop from being called. And we always exited with 0. Now the exit status is determined by DesignerStarter#launchGui and used correctly. Platform#exit triggers the shutdown of JavaFx. After that is done execution continues in DesignerStarter after Application#launch.

This avoids "your app is not responding" errors, while the git command is waiting for
password input to load the gpg key for commit signing.

This also reverts the fix from 7cc0463:
Calling System#exit immediately after Platform#exit prevents Application#stop from
being called. And we always exited with 0. Now the exit status is determined by
DesignerStarter#launchGui and used correctly. Platform#exit triggers the shutdown
of JavaFx. After that is done execution continues in DesignerStarter after
Application#launch.
@adangel adangel added this to the 7.0.0 milestone Jun 9, 2023
adangel added a commit that referenced this pull request Sep 28, 2023
adangel added a commit that referenced this pull request Sep 28, 2023
Perform the persistence asynchronously to not block the main (UI) thread #69
@adangel adangel merged commit 127367b into pmd:master Sep 28, 2023
@adangel adangel deleted the async-exit branch September 28, 2023 12: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