On Fri, 2017-03-24 at 13:44 -0700, Eric Anholt wrote:

> This is the only functional change I spotted: SaveScreen is now going to
> return FALSE instead of TRUE.  Given that we haven't actually blanked
> the screen, this seems like an appropriate return value.

I don't think this is a functional change:

> > @@ -645,34 +614,7 @@ KdCloseScreen(ScreenPtr pScreen)
> >  static Bool
> >  KdSaveScreen(ScreenPtr pScreen, int on)
> >  {
> > -    KdScreenPriv(pScreen);
> > -    int dpmsState;
> > -
> > -    if (!pScreenPriv->card->cfuncs->dpms)
> > -        return FALSE;
> > -

->dpms was always null so we were always returning false anyway.

- ajax
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to