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

Control-C may leave a defunct process #3456

Closed
mattulbrich opened this issue Apr 14, 2024 · 3 comments
Closed

Control-C may leave a defunct process #3456

mattulbrich opened this issue Apr 14, 2024 · 3 comments
Assignees
Milestone

Comments

@mattulbrich
Copy link
Member

Description

When starting KeY from the command line, pressing ctrl-c prior to the main window's appearance leaves KeY in a defunct state that can only be killed from outside. Pressing ctrl-c again does not help.

Reproducible

sometimes.

It seems the time when ctrl-c is pressed plays a role. It seems that KeY must be in a state where the signal is registered, yet the exit action does not have its full effect yet.

Steps to reproduce

Start KeY from the command line (e.g. using the shadow jar) and press ctrl-c before the window shows up.

It should terminate, but instead it only prints WARN WindowUserInterfaceControl - Caught SIGINT, exiting...to the console and gets stuck.

Additional information

The signal handler seems to have been introduced in 2ff6a37.


@mattulbrich
Copy link
Member Author

Nice issue number btw 😄

@wadoon
Copy link
Member

wadoon commented Apr 28, 2024

In #3451, I have already removed this part. This class is part of the module jdk.unsupported. I think the authors want to tell us something with this name. 🤔

@wadoon wadoon added this to the v2.14.0 milestone Apr 28, 2024
wadoon added a commit that referenced this issue Jul 20, 2024
wadoon added a commit that referenced this issue Jul 20, 2024
@wadoon wadoon modified the milestones: v2.14.0, v2.12.2 Jul 20, 2024
wadoon added a commit that referenced this issue Aug 4, 2024
@WolframPfeifer
Copy link
Member

Closed by #3499.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants