Already effectively required by use of XNFcallocarray() introduced in
xorg/xserver@b96dc999 - xserver-1.18.0, released in Nov. 2015.

Allows dropping remnants of code for XAA and ISA

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <!9>

xnfcalloc is just an alias for XNFcallocarray() that doesn't seem to serve
any practical purpose, so it can go away once all drivers stopped using it.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <!7>

xnfalloc is just an alias for XNFalloc() that doesn't seem to serve
any practical purpose, so it can go away once all drivers stopped using it.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <!7>

The Xorg headers provide their own versions of strlcat, strlcpy, and
timingsafe_memcmp for platforms that don't have them in libc yet, but
rely on configure to set HAVE_* defines to determine if they should be
defined in the headers.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

neo_driver.c: In function ‘NEOScreenInit’:
neo_driver.c:1638:14: warning: declaration of ‘ret’ shadows a previous local [-Wshadow]
 1638 |         Bool ret = FALSE;
      |              ^~~
neo_driver.c:1460:9: note: shadowed declaration is here
 1460 |     int ret;
      |         ^~~

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

AC_PROG_LIBTOOL was replaced by LT_INIT in libtool 2 in 2008,
so it's time to rely on it.

Clears autoconf warnings:

configure.ac:44: warning: The macro 'AC_PROG_LIBTOOL' is obsolete.
configure.ac:44: You should run autoupdate.
aclocal.m4:3465: AC_PROG_LIBTOOL is expanded from...
configure.ac:44: the top level

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

neo_driver.c: In function ‘NEOPreInit’:
neo_driver.c:892:9: warning: this statement may fall through
  [-Wimplicit-fallthrough=]
      if (nPtr->NeoChipset != NM2070)
         ^
neo_driver.c:894:2: note: here
  default:
  ^~~~~~~

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

neo_driver.c: In function ‘NEOGetRec’:
neo_driver.c:464:5: warning: this ‘if’ clause does not guard...
  [-Wmisleading-indentation]
     if (pScrn->driverPrivate == NULL)
     ^~
neo_driver.c:466:9: note: ...this statement, but the latter is
  misleadingly indented as if it were guarded by the ‘if’
         return TRUE;
         ^~~~~~

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Found by using:
    codespell --builtin clear,rare,usage,informal,code,names

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Matthieu Herrb authored 6 years ago and Kevin Brace committed 6 years ago

Silences clang warnings.

Signed-off-by: Matthieu Herrb <matthieu@herrb.eu>
Reviewed-by: Walter Harms <wharms@bfs.de>

Signed-off-by: Kevin Brace <kevinbrace@gmx.com>

Signed-off-by: Kevin Brace <kevinbrace@gmx.com>

Signed-off-by: Kevin Brace <kevinbrace@gmx.com>

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Mihail Konev authored 8 years ago and Peter Hutterer committed 8 years ago

Signed-off-by: Mihail Konev <k.mvc@ya.ru>

Emil Velikov authored 10 years ago and Peter Hutterer committed 8 years ago

Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
fall-outs, when they contain space.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Syncs the invocation of configure with the one from the server.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

Alan Coopersmith authored 10 years ago and Peter Hutterer committed 8 years ago

See http://people.gnome.org/~walters/docs/build-api.txt



Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Alan Coopersmith authored 10 years ago and Peter Hutterer committed 8 years ago

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Signed-off-by: Julien Cristau <jcristau@debian.org>

xf86GetOptValString was changed to return const char *.

Signed-off-by: Julien Cristau <jcristau@debian.org>

Signed-off-by: Dave Airlie <airlied@redhat.com>

I plan to remove the Weak functions from a future server

Signed-off-by: Adam Jackson <ajax@redhat.com>

Signed-off-by: Matt Turner <mattst88@gmail.com>

Matthieu Herrb authored 11 years ago and Matt Turner committed 11 years ago

Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=434468


Signed-off-by: Matthieu Herrb <matthieu.herrb@laas.fr>
Signed-off-by: Matt Turner <mattst88@gmail.com>

Signed-off-by: Adam Jackson <ajax@redhat.com>

DGA unneeded since b6633ee7, others probably never needed.

Signed-off-by: Matt Turner <mattst88@gmail.com>

Signed-off-by: Matt Turner <mattst88@gmail.com>

Signed-off-by: Matt Turner <mattst88@gmail.com>

The commit that makes XAA optional left a use of NEO_Sync while putting
the body inside an #ifdef HAVE_XAA_H block, which caused undefined
symbol errors when loading the driver. NEO_Sync doesn't do anything
without XAA, so don't bother calling it without XAA.

Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=434468


Reviewed-by: Alex Deucher <alexdeucher@gmail.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

Signed-off-by: Dave Airlie <airlied@redhat.com>

The 'damage' member is an ancient compatibility hack I want to destroy.

Signed-off-by: Adam Jackson <ajax@redhat.com>