Skip to content

fix: terminate podcast on program exit#1300

Merged
kaamui merged 1 commit intoOpenBoard-org:devfrom
letsfindaway:fix-terminate-podcast
Jan 30, 2026
Merged

fix: terminate podcast on program exit#1300
kaamui merged 1 commit intoOpenBoard-org:devfrom
letsfindaway:fix-terminate-podcast

Conversation

@letsfindaway
Copy link
Copy Markdown
Collaborator

@letsfindaway letsfindaway commented Aug 3, 2025

  • When OpenBoard is closed while a podcast is running the application is hanging waiting for a thread to finish.
  • Cleanly terminate the podcast recording before closing the application.

Fixes letsfindaway#223

Note: The lastWindowClosed signal is not emitted from the application when terminating OpenBoard. We therefore now connect to UBMainWindow::closeEvent_Signal. This also affects other locations, but there we always have another way in place to detect termination, so I did not touch them.

- when OpenBoard is closed while a podcast is running
  the application is hanging waiting for a thread to finish
- cleanly terminate the podcast recording before closing
  the application

Signed-off-by: letsfindaway <me@letsfindaway.de>
@letsfindaway letsfindaway force-pushed the fix-terminate-podcast branch from 2a3236f to c7809d9 Compare December 3, 2025 16:20
@letsfindaway
Copy link
Copy Markdown
Collaborator Author

I now added a timer to avoid endless waiting.

Also note that a similar construct using processEvents() is also in the destructor of the UBPersistenceManager and seems to work for several years without problem.

@kaamui : If you think it would be useful I can also move the target branch to 1.7.4-dev-

@kaamui kaamui merged commit 61fa15a into OpenBoard-org:dev Jan 30, 2026
@letsfindaway letsfindaway deleted the fix-terminate-podcast branch January 30, 2026 09:52
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.

[Bug] Podcast recording not cleanly terminated

2 participants