Creating libX11 symlinks breaks our builds in clean environments. Drop this
feature for now.

It must be rewritten to only create the symlinks after everything has been
built.

is_numeric is only needed if X11_t is defined.

Silences "warning: ‘is_numeric’ defined but not used [-Wunused-function]"

warning: "XTRANS_SEND_FDS" is not defined [-Wundef]

This define came in via the recent xtrans update.

warning: format ‘%p’ expects argument of type ‘void *’

This lifts xtrans to the state of this commit:
    commit 560d7550e23e9b14056b4a9b2569c2f256015f8a
    Author: Jeremy Huddleston Sequoia <jeremyhu@apple.com>
    Date:   Sat Sep 10 22:09:51 2016 -0700

        Update strlcpy macro check to also check HAVE_STRLCPY

        xorg-server moved from HAS_STRLCPY to HAVE_STRLCPY in 2011

        cf-xserver: d829a7c5cb42c979b58f3547136df5b05d906423
Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>

Remove CLTS code

Never been used, as far as I can tell.
Reviewed-by: Keith Packard <keithp@keithp.com>
Signed-off-by: Adam Jackson <ajax@redhat.com>
Backported-to-NX-by: Ulrich Sibiller <uli42@gmx.de>

this simplyfies updating to Xorg/xtrans upstream

"""

It's simply obsolete, sloppy, compiler namespace pollution.  The
compiler is not allowed to predefine symbols that might conflict with
ordinary identifiers.  For backwards compatibility gcc currently
predefines i386 when compiling for x86 32-bit (but not 64-bit), but that
will go away.  It is also not defined if you specify -ansi when invoking
the compiler, because then it is seriously standards compliant.  Other
compilers shouldn't define it either.  Correct code shouldn't rely on it
being defined.  However __i386__ is safe and proper.

"""
Backported-to-NX-by: Ulrich Sibiller <uli42@gmx.de>

empty lines and trailing whitespace

If your OS doesn't have sysconf(3), then life is already hard for you.
Backported-to-NX-by: Ulrich Sibiller <uli42@gmx.de>

just as Xorg upstream did

although upstream still includes it.

never used

Create libX11* link in exports/lib. This simplifies running nxagent
from the build directory.

This also adds a clean rule to remove the libX11 links as well as the
libNX_X11 links which was not done before.

Fixes ArcticaProject/nx-libs#234

 Fixes ArcticaProject/nx-libs#250.

This was introduced in 8ea762f.
Reported-by: Julien Cristau <jcristau@debian.org>
Signed-off-by: Emilio Pozuelo Monfort <pochu@debian.org>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

LRGB.c: In function ‘LINEAR_RGB_InitSCCData’:
LRGB.c:798:1: warning: label ‘FreeBlueTblElements’ defined but not used [-Wunused-label]
 FreeBlueTblElements:
 ^

Fixes: warning: variable 'req' set but not used [-Wunused-but-set-variable]
       by marking req _X_UNUSED
	Solution was discussed on xorg-devel ML
       Peter Hutter, Alan Coopersmith
        Re: [PATCH libX11 3/5] fix: warning: pointer targets in passing argument 2 of '_XSend' differ in signedness [-Wpointer-sign]

Signed-off-by: harms wharms@bfs.de

fix: warning: pointer targets in passing argument 2 of '_XSend' differ in signedness [-Wpointer-sign]

fix signess warning by casting
Signed-off-by: walter harms <wharms@bfs.de>

simplify code
Signed-off-by: walter harms <wharms@bfs.de>

remove stray extern
Signed-off-by: walter harms <wharms@bfs.de>

XlibInt.c: In function ‘_XIOError’:
XlibInt.c:3750:1: warning: ‘noreturn’ function does return [enabled by default]
 }

After fixing the above a new warning came up, which is also fixed now:

XlibInt.c: In function ‘_XIOError’:
XlibInt.c:3750:1: warning: control reaches end of non-void function [-Wreturn-type]
 }