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

Ghost windows are not closing + system cleanup feature #461

Closed
labordep opened this issue Mar 5, 2024 · 4 comments
Closed

Ghost windows are not closing + system cleanup feature #461

labordep opened this issue Mar 5, 2024 · 4 comments

Comments

@labordep
Copy link
Contributor

labordep commented Mar 5, 2024

Hi,

Related to #457, we have always ghost windows that are not closing correctly (SDL2).
This is not easy to reproduce at each time, I think we need to use a clean safety code.
Do you have the code below in a Bloc "system cleanup" feature ?

BlOSWindowSDL2Host universe closeSpaces.
BlOSWindowSDL2Host
	stop;
	start.
BlMorphicWindowHost universe closeSpaces.
BlMorphicWindowHost
	stop;
	start 

This code, extracted from Toplo, is very useful for cleaning up ghost windows. If it exists, could we integrate this code when cleaning up the image via the system menu?
image

Thanks :)

Config:

  • Windows 10 Pro
  • Pharo-11.0.0+build.725
  • VM 10.1.0.0
@tinchodias
Copy link
Collaborator

In addition, this is a script I used once to force destroying SDL windows:

"Destroy all SDL windows except the Pharo IDE (assuming its title ends with '.image')"
SDL_Window allInstances
	reject: [ :each | each isNull or: [ each title endsWith: '.image' ] ]
	thenDo: [ :each | each hide; destroy ]

@labordep
Copy link
Contributor Author

labordep commented Mar 6, 2024

After executing twice the test on #463 I have my ghost window.
After, I execute this code:

BlOSWindowSDL2Host universe closeSpaces.
BlOSWindowSDL2Host
	stop;
	start.
BlMorphicWindowHost universe closeSpaces.
BlMorphicWindowHost
	stop;
	start 

And after a Do Image Cleanup I got this error:
image

@labordep
Copy link
Contributor Author

labordep commented Mar 6, 2024

@tinchodias I wrote a branch here #464

@tinchodias
Copy link
Collaborator

The PRs are merged so I guess we can close this issue

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

2 participants