[MPlayer-dev-eng] [PATCH] Enable energy saving option and screensaver mode when the video is paused.

[Ferenc Czumbil]

On 01/12/14 22:40, Reimar Döffinger wrote:
> On Mon, Dec 01, 2014 at 05:57:47PM +0200, Ferenc Czumbil wrote:
>> On 29/11/14 23:58, Hans-Dieter Kosch wrote:
>>> Reimar Döffinger wrote:
>>>
>>>> On Thu, Nov 27, 2014 at 07:09:26PM +0200, Ferenc Czumbil wrote:
>>>>> Hi,
>>>>>
>>>>> Currently, by default, energy saving and the screensaver is disabled
>>>>> when mplayer starts and is re-enabled on exit.
>>>>>
>>>>> This re-enables them when the video is paused. Fixes bug #2112
>>>> I'm not sure everyone likes this behaviour.
>>>> It might be better to add an option to choose the behaviour.
>>> I'd also vote for a selectable behaviour. A user may want to observe a
>>> picture of a paused stream, and then annoyingly the screensaver
>>> interferes...
>>>
>> Should I add a new separate option? Or would it be better to have a single
>> option for selecting the desired behaviour? Something like:
>>
>> -stop-xscreensaver true|false|on_playback
> The idea sounds nice but I see two issues:
> 1) It will probably be somewhat more complex
> 2) It will probably break existing config files
>
> So I'd expect a separate option to be the better solution.
Fair enough. It was a long time ago when I did some programming and I'm 
not familiar with the source code of mplayer, but the attached patch 
seams to work (at least for me). I'm open to any further suggestions.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: xsaver.patch
Type: text/x-patch
Size: 2676 bytes
Desc: not available
URL: <https://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20150122/1ce4c425/attachment.bin>