On Thu, Mar 11, 2021 at 03:36:03PM +0100, Alex Raschi wrote: > Ping > > There was an extra space after 'noff', below is the updated patch. > > On Tue, Feb 23, 2021 at 11:11:21AM +0100, Alex Raschi wrote: > > Hi, > > > > I noticed that xlock does not allow values between 30 and 0 for options > > -dpms{standby,suspend,off}, the man page suggests that all values >= 0 > > are accepted. I use the blank mode and i would like to set a lower > > timeout so i made a patch to remove the limit. > > > > ok?
Hi, sorry I didn't pay attention your mail initially. I've commited the patch below. Thanks. > > Index: app/xlockmore/xlock/xlock.c > =================================================================== > RCS file: /cvs/xenocara/app/xlockmore/xlock/xlock.c,v > retrieving revision 1.2 > diff -u -p -r1.2 xlock.c > --- app/xlockmore/xlock/xlock.c 26 Nov 2006 17:13:23 -0000 1.2 > +++ app/xlockmore/xlock/xlock.c 20 Feb 2021 15:37:47 -0000 > @@ -624,7 +624,6 @@ extern int XHPEnableReset(Display * dsp > > #endif > #ifdef USE_DPMS > -#define MIN_DPMS 30 /* 30 second minimum */ > #if 1 > #include <X11/Xmd.h> > #include <X11/Xdmcp.h> > @@ -1114,9 +1113,9 @@ SetDPMS(Display * display, int nstandby, > DPMSSetTimeouts(display, standby, suspend, off); > else > DPMSSetTimeouts(display, > - (nstandby <= 0 ? 0 : (nstandby > > MIN_DPMS ? nstandby : MIN_DPMS)), > - (nsuspend <= 0 ? 0 : (nsuspend > > MIN_DPMS ? nsuspend : MIN_DPMS)), > - (noff <= 0 ? 0 : (noff > MIN_DPMS ? > noff : MIN_DPMS))); > + (nstandby <= 0 ? 0 : nstandby), > + (nsuspend <= 0 ? 0 : nsuspend), > + (noff <= 0 ? 0 : noff)); > } > } -- Matthieu Herrb