From 0041bc0fae0b42ca4fa4ec7aca1d6bcd66000a6a Mon Sep 17 00:00:00 2001 From: rob Date: Tue, 30 Dec 2025 04:00:38 -0400 Subject: [PATCH] Fix Qt cleanup warning on dialog close MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add closeEvent that waits for thread pool workers to finish before closing, preventing "shared QObject deleted directly" warning. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- src/ramble/dialog.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/ramble/dialog.py b/src/ramble/dialog.py index 3d71e3f..81d36e1 100644 --- a/src/ramble/dialog.py +++ b/src/ramble/dialog.py @@ -428,6 +428,11 @@ class RambleDialog(QDialog): self.result = RambleResult(summary=summary, fields=out_fields) self.accept() + def closeEvent(self, event): + """Wait for background workers to finish before closing.""" + self.thread_pool.waitForDone(1000) # Wait up to 1 second + super().closeEvent(event) + def open_ramble_dialog( *,