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

[Reimar Döffinger]

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.