xserver: Branch 'master'
David Nusinow
dnusinow at speakeasy.net
Mon Jan 7 17:54:14 PST 2008
On Mon, Jan 07, 2008 at 09:00:14AM +0100, Michel Dänzer wrote:
>
> On Sun, 2008-01-06 at 13:37 -0800, David Nusinow wrote:
> > hw/xfree86/common/xf86DPMS.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > New commits:
> > commit 260505e3c5a18044e97d31ea3bcc0955e46335c8
> > Author: David Nusinow <dnusinow at debian.org>
> > Date: Sun Jan 6 16:37:13 2008 -0500
> >
> > Log enabling of DPMS even when it's not set in xorg.conf
> >
> > diff --git a/hw/xfree86/common/xf86DPMS.c b/hw/xfree86/common/xf86DPMS.c
> > index 43efb8e..a23d81d 100644
> > --- a/hw/xfree86/common/xf86DPMS.c
> > +++ b/hw/xfree86/common/xf86DPMS.c
> > @@ -79,7 +79,6 @@ xf86DPMSInit(ScreenPtr pScreen, DPMSSetProcPtr set, int flags)
> > && !DPMSDisabledSwitch)
> > DPMSEnabled = TRUE;
> > xf86MarkOptionUsed(DPMSOpt);
> > - xf86DrvMsg(pScreen->myNum, X_CONFIG, "DPMS enabled\n");
> > } else if (DPMSEnabledSwitch) {
> > if (!DPMSDisabledSwitch)
> > DPMSEnabled = TRUE;
> > @@ -88,6 +87,8 @@ xf86DPMSInit(ScreenPtr pScreen, DPMSSetProcPtr set, int flags)
> > else {
> > pDPMS->Enabled = defaultDPMSEnabled;
> > }
> > + if (pDPMS->Enabled)
> > + xf86DrvMsg(pScreen->myNum, X_CONFIG, "DPMS enabled\n");
>
> Please don't use X_CONFIG when it's not due to xorg.conf.
Crap, sorry about that. Thanks for catching this stupid copy 'n paste
idiocy. I'll push a patch that uses X_CONFIG when it's in xorg.conf and
X_INFO when not.
- David Nusinow
More information about the xorg
mailing list