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

Destroy desktop integration at correct place #18108

Merged
merged 1 commit into from Nov 30, 2022

Conversation

glassez
Copy link
Member

@glassez glassez commented Nov 30, 2022

Otherwise it is destructed in QObject destructor, i.e. after GUI application is already destructed.
This can be related to some problems with system tray icon.

Closes #18093.

@glassez glassez added GUI GUI-related issues/changes Hotfix Fix some bug(s) introduced by recently merged commit(s) labels Nov 30, 2022
@glassez glassez added this to the 4.6.0 milestone Nov 30, 2022
@glassez glassez requested review from sledgehammer999 and a team November 30, 2022 06:46
@sledgehammer999
Copy link
Member

Otherwise it is destructed in QObject destructor, i.e. after GUI application is already destructed.

Great deduction of the possible problem. There is one user who tested it and confirmed that it works.

I also asked in the other 2 bug reports for someone to test.

@glassez glassez merged commit 25ea0d2 into qbittorrent:master Nov 30, 2022
@glassez glassez deleted the destruct-tray branch November 30, 2022 17:10
glassez added a commit to glassez/qBittorrent that referenced this pull request Nov 30, 2022
Otherwise it is destructed in QObject destructor, i.e. after GUI application is already destructed.
This can be related to some problems with system tray icon.

PR qbittorrent#18108.
Closes qbittorrent#18093.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI GUI-related issues/changes Hotfix Fix some bug(s) introduced by recently merged commit(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

qbittorrent-4.5.0 gets into some infinite loop upon exit and uses 100% cpu
4 participants