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

Trac#3011 Handle WM_ENDSESSION on Windows to shutdown cleanly #155

Open
wants to merge 1 commit into
base: 1.3-stable
from

Conversation

Projects
None yet
2 participants
@cas--
Contributor

cas-- commented May 11, 2017

This is not ready for merge however it is working code and will catch the wm_queryendsession messages. The next step is to handle this correctly so Deluge is given enough time to save data to disc as the default 5 secs is probably not enough.

See trac ticket for more details.

add code to catch endsession message
 * only register message handling on first show
@DjLegolas

This comment has been minimized.

Show comment
Hide comment
@DjLegolas

DjLegolas Dec 18, 2017

Contributor

I wrote a handler in ef6ca96.
On WM_QUERYENDSESSION message, Deluge will register itself as a Shutdown-Blocker.
Next, when receiving WM_ENDSESSION message, Deluge should have 30 seconds to save all the data - depending on whether critical shutdown or not.

But the lack of a proper tool for testing this makes me feel like the is something wrong with my work.

Contributor

DjLegolas commented Dec 18, 2017

I wrote a handler in ef6ca96.
On WM_QUERYENDSESSION message, Deluge will register itself as a Shutdown-Blocker.
Next, when receiving WM_ENDSESSION message, Deluge should have 30 seconds to save all the data - depending on whether critical shutdown or not.

But the lack of a proper tool for testing this makes me feel like the is something wrong with my work.

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