Paused video will not allow monitor energy saving mode on windows

The video player should only block monitor energy saving mode when playing visual elements ie a movie. (You do not need the screen when playing music)

I've been waiting for several version on somebody to fix this annoying problem. Now i'm talking the time to point out the issue myself.

When a video is paused in vlc it should allow windows to turn off the screen.

added Component::Interface: Qt Difficulty::easy Severity::normal Type::bug Version::master git labels

changed milestone to %Bugs paradize

assigned to @fenrir and unassigned @jbk

added Component::Video: Direct3D / Win32 label and removed Component::Interface: Qt label

Works for me. VLC version?

Official VLC (7zip package) v2.0.4 Twoflower

I downloaded it from here: http://www.videolan.org/vlc/download-windows.html

I have two computers. VLC blocks screen sleep mode on both of them. I have windows 7 x64 on both computers.

Original author: https://www.google.com/accounts/o8/id?id=aitoawnde3ujxbdx6lky9nacu2z5ebn2thtmbaq

Have any progress been made on this issue yet ? Should be quite an easy patch.

FYI: The execution state needs to be updated when the video is paused to:

SetThreadExecutionState( ES_CONTINUOUS );

or if you want to keep the system from going to sleep:

SetThreadExecutionState( ES_SYSTEM_REQUIRED | ES_CONTINUOUS );

When the video is unpaused:

SetThreadExecutionState( ES_DISPLAY_REQUIRED | [ES_SYSTEM_REQUIRED |] ES_CONTINUOUS );

Of course there should be a user setting to allow control of this. Not everyone would want the system to be kept alive when the video is paused.

I would have made a patch myself but i'm unfamiliar with vlc code. My first impression of it during a quick look of the player part is messy. Reminds me of my kids room.

Original author: https://www.google.com/accounts/o8/id?id=aitoawnde3ujxbdx6lky9nacu2z5ebn2thtmbaq

Can somebody please fix this bug already

Original author: fish sauce

unassigned @fenrir

Yeah. Volunteer welcome.

Volunteer's asking for volunteers. No wonder nothing gets done.

I know what code to add. I just do not know where to put it.

The code were 'SetThreadExecutionState' is called seem to be a display module. It does not give a clear indication of where the player respond to the pause / play events.

As shown above. You have to do that in the pause and unpause events.

That is the patch. I have looked at the code and i have not a clue of where to add this code.

Put this in the paused event: 'SetThreadExecutionState( ES_SYSTEM_REQUIRED | ES_CONTINUOUS );'

Put this in the unpaused event: 'SetThreadExecutionState( ES_DISPLAY_REQUIRED | ES_SYSTEM_REQUIRED | ES_CONTINUOUS );'

I could write the code and make a patch you just have to tell me were to to it.

So the volunteer needs a more experienced vlc volunteer to help.

Original author: fish sauce

See #3724 (closed) too.

#3724 (closed) is now fixed. Please try 3.0

added Status::duplicate label

closed

mentioned in issue #8908 (closed)

mentioned in issue #11932 (closed)