As reported by static analyzer.

Attributes ArcticaProject/nx-libs#905

Use the "xorg_" prefix as everywhere else, not "Xorg_".

Also add original code to make NX changes obvious. Also change some
lines to match original code.

and use DEBUG like everywhere else. There were already some TEST lines
using this scheme.

Also indent DEBUG and TEST lines to make the code more readable

make compilation work with -DNX_DEBUG_INPUT

NX_WAKEUP had been non-effective and its effect had been tied to NX_TRANS_WAKEUP.

1. indent ifdefs
2. add some logic at start that unset NX_TRANS_WAKEUP and
NX_TRANS_DEBUG if NX_TRANS_SOCKET is unset. This way we only have to
check for one macro and not all three.
3. remove redundant macro checks
4. decouple debug printfs by always running the original code and adding the
   identical check in #ifdef NX_TRANS_DEBUG

and remove obsolete NX_SPLASH

Instead of hardcoding the keystrokes in Dialog.h determine the
currently configured keystrokes for the action and insert them into
the dialog strings.

Fixes ArcticaProject/nx-libs#438

and use True/False instead of 1/0

This change ensures Literals.h is always referenced during
compilation, effectively hiding it from "unused file" detection we do
occasionally to clean up the source code.

Derived from

  commit 978f3b496b9951ee8120a0efcc5cd12503e26770
  Author: Roland Mainz <roland.mainz@nrubsig.org>
  Date:   Sat Mar 5 21:38:29 2005 +0000

    xc/programs/Xserver/Imakefile
    xc/programs/Xserver/hw/xnest/Imakefile
    //bugs.freedesktop.org/show_bug.cgi?id=2653) attachment #2020
        (https://bugs.freedesktop.org/attachment.cgi?id=2020): Cleanup Xnest
        usage of the DPMS dummy stub functions from dpmsstubs.c instead of
        using it's own copy of these functions.

Fixes ArcticaProject/nx-libs#901

File was missing in Imakefile. We are not using the ffs function, but
nevertheless it's a part of dix.

Backport of this xorg-xserver commit:

  commit 75c51c67b340548286efd41a53882e2acaf74ab5
  Author: Alan Coopersmith <alan.coopersmith@sun.com>
  Date:   Thu Jun 18 09:49:12 2009 -0700

    Clarify use of and need for mffs vs. ffs
Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>

Follow-up for 2a5364d5

See ArcticaProject/nx-libs#263 for discussion

Instead of 17 XFillPolygon() calls we now use 5.

Was wrong when scale was different from 1

It is not needed elsewhere.

They are not used from the outside.

was offset to the right and downwards before

Fixes ArcticaProject/nx-libs#892

The problem we are fixing (or better "working around") here has also
been seen in XQuartz, but no-one has found the reason yet.

Point to bugreport so that anyone stumbling over this in future can
check if there's a better fix and possibly implement that one.

Backport of this xorg-xserver commit:

  commit cfd3443fe81685e91a53063dee58e24a5684dc29
  Author: Peter Hutterer <peter.hutterer@who-t.net>
  Date:   Wed Jan 28 13:34:10 2009 +1000

    Xext: rename saver's EventType to SaverEventType.

    Avoid namespace clashing with the internal events.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>