[MPlayer-dev-eng] [PATCH] Make disabling DPMS optional

Paul TT paultt at hackerjournal.it
Fri Nov 3 20:09:53 CET 2006


On Fri, 3 Nov 2006 16:52:21 +0100
Holger Weiss <holger at CIS.FU-Berlin.DE> wrote:

> * RC <rcooley at spamcop.net> [2006-11-03 03:22]:
> > Holger Weiss <holger at CIS.FU-Berlin.DE> wrote:
> > > MPlayer disables DPMS on startup and re-enables it during exit (if
> > > DPMS was enabled in the first place); this behaviour is hardcoded.
> >
> > You can easily disable DPMS in config.h.  IMHO, it's not really
> > worth having a command-line option.
> 
> At least for us, an option would definitely be more convenient, as we
> wouldn't be forced to build MPlayer from source.  Also, the binary is
> exported to quite a few workstations via NFS and I'm not sure whether
> all users always want the same behaviour.
> 
> > In any case, I'd certainly pick better option names...  Perhaps
> > -(no)dpms ?
> 
> I thought about "-(no)dpms" myself, but found it a bit ambiguous: Does
> "-nodpms" mean "disable DPMS" or "don't disable DPMS"?  If you tell me
> which way round to do it, I'll send an updated pacth :-)

i read it this way:
no dpms to me means something like 'don't take care of dpms', so don't
disable. anyway i found totally useless to have the /true/ part, to let
mplayer disable it.
i think -nodpms, meaning 'don't take care of dpms is enough
MHO



More information about the MPlayer-dev-eng mailing list