Fix shadow warning:

xkbLEDs.c: In function ‘XkbFindSrvLedInfo’:
xkbLEDs.c:633:19: warning: declaration of ‘sli’ shadows a previous local [-Wshadow=compatible-local]
  XkbSrvLedInfoPtr sli;
                   ^~~
xkbLEDs.c:629:18: note: shadowed declaration is here
 XkbSrvLedInfoPtr sli;
                  ^~~

Backport of this xorg-xserver commit:

  commit d8eb9b5faa2d2c9350bdaffef8749a9cac3c87b8
  Author: Alan Coopersmith <alan.coopersmith@oracle.com>
  Date:   Mon Dec 12 16:49:34 2011 -0800

    XkbFindSrvLedInfo: remove extraneous name-clashing sli variable

    Variable is already defined outside the outer if statement, and
    there's no need to redefine inside the if statement.

    No point in setting sli before if (dev->kbdfeed->xkb_sli==NULL)
    check - if check is true, we immediately set it, if check is false,
    we immediately return without further reference or use of it.

    The one thing we do with it inside the inner if statement is store
    an allocation in it for a brief moment before writing to the final
    destination, which is immediately returned to the caller.

    In short, there's no benefit to the variable at all in this block,
    it just gives the optimizer more code to figure out how to omit.

    Fixes gcc warning:
    xkbLEDs.c: In function 'XkbFindSrvLedInfo':
    xkbLEDs.c:683:19: warning: declaration of 'sli' shadows a previous local
    xkbLEDs.c:679:18: warning: shadowed declaration is here
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

xkb.c: In function ‘ProcXkbSetCompatMap’:
xkb.c:2601:20: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local]
  register unsigned i,bit;
                    ^
xkb.c:2529:23: note: shadowed declaration is here
     register unsigned i,bit;
                       ^
xkb.c:2601:22: warning: declaration of ‘bit’ shadows a previous local [-Wshadow=compatible-local]
  register unsigned i,bit;
                      ^~~
xkb.c:2529:25: note: shadowed declaration is here
     register unsigned i,bit;
                         ^~~

Backport of this xorg upstream commit:

  commit 344eea237fc07dedfd733d14f95ed0ad26bb5f81
  Author: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
  Date:   Tue Oct 16 02:04:34 2012 -0500

    xkb: fix shadow warnings
Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

  commit 424dbde891486ad6a6c00c61a334031ff18f5556
  Author: Alan Coopersmith <alan.coopersmith@oracle.com>
  Date:   Fri Dec 9 10:08:55 2011 -0800

    CheckForEmptyMask does not need to declare int n twice

    Just use the existing n variable again in the ARGB_CURSOR loop
    instead of creating another one.

    Fixes gcc -Wshadow warning:
    cursor.c: In function 'CheckForEmptyMask':
    cursor.c:155:6: warning: declaration of 'n' shadows a previous local
    cursor.c:146:9: warning: shadowed declaration is here
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

xp must be declared outside the while loop!

xp was reset in each interation despite being increased at the
iteration's end!

To protect against this happening again replace the while loop by a
for loop that references xp.

Bug introduced by d94f4727

Pixmap.c: In function ‘nxagentDisconnectAllPixmaps’:
Pixmap.c:677:19: warning: declaration of ‘r’ shadows a previous local [-Wshadow=compatible-local]
   for (int i = 0, r = 1; i < MAXCLIENTS; r = 1, i++)
                   ^
Pixmap.c:652:7: note: shadowed declaration is here
   int r = 1;
       ^
Pixmap.c: In function ‘nxagentReconnectAllPixmaps’:
Pixmap.c:840:19: warning: declaration of ‘result’ shadows a previous local [-Wshadow=compatible-local]
   for (int i = 0, result = 1; i < MAXCLIENTS; result = 1, i++)
                   ^~~~~~
Pixmap.c:807:8: note: shadowed declaration is here
   Bool result = 1;
        ^~~~~~

Display.c: In function ‘nxagentInitAndCheckVisuals’:
Display.c:2273:10: warning: declaration of ‘matched’ shadows a previous local [-Wshadow=compatible-local]
     bool matched = false;
          ^~~~~~~
Display.c:2258:8: note: shadowed declaration is here
   bool matched;
        ^~~~~~~
Display.c:2258:8: warning: unused variable ‘matched’ [-Wunused-variable]

Events.c: In function ‘nxagentDispatchEvents’:
Events.c:1241:21: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local]
           ScreenPtr pScreen = nxagentScreen(X.xbutton.window);
                     ^~~~~~~
Events.c:827:13: note: shadowed declaration is here
   ScreenPtr pScreen = NULL;
             ^~~~~~~
Events.c:1383:19: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local]
         ScreenPtr pScreen = nxagentScreen(X.xmotion.window);
                   ^~~~~~~
Events.c:827:13: note: shadowed declaration is here
   ScreenPtr pScreen = NULL;
             ^~~~~~~

Events.c: In function ‘nxagentDispatchEvents’:
Events.c:1383:19: warning: declaration of ‘pScreen’ shadows a previous local [-Wshadow=compatible-local]
         ScreenPtr pScreen = nxagentScreen(X.xmotion.window);
                   ^~~~~~~
Events.c:827:13: note: shadowed declaration is here
   ScreenPtr pScreen = NULL;
             ^~~~~~~

this was harmless but not nice nevertheless.

Clipboard.c: In function ‘nxagentConvertSelection’:
Clipboard.c:1850:9: warning: declaration of ‘i’ shadows a previous local [-Wshadow=compatible-local]
     int i = nxagentFindCurrentSelectionIndex(selection);
         ^
Clipboard.c:1736:7: note: shadowed declaration is here
   int i = nxagentFindCurrentSelectionIndex(selection);
       ^

Revert a wrong change that misteriously slipped in via the comment
reformatting commit 0133d461.

Fixes ArcticaProject/nx-libs#893

Using XlibAtom here is wrong. On a 64bit system the size of this type
is 8 bytes (64bit). At this stage we do not read Xlib datatypes but
wire datatypes. Over the wire an Atom is always 32bit, so we need to
use CARD32 here.

Fixes ArcticaProject/nx-libs#894

 Regression fix for 427b7b97. As suggested
 by Ulrich Sibiller.

 Fixes ArcticaProject/nx-libs#891.

Attributes GH PR #880: https://github.com/ArcticaProject/nx-libs/pull/880

nxagentBitmapBitOrder never existed and should really be
BitmapBitOrder. It was accidently replaced in
523009b9

used the wrong macro for length determination

nxagentPrintError is derived from XlibInt.c:_XprintDefaultError. Let's
add missing code and mark NX changes.

Backport of this xorg-xserver commit:

  commit a82e6efb7b9b2ab9a1597b002f375c5ee105e7f5
  Author: Adam Jackson <ajax@redhat.com>
  Date:   Wed Jul 16 14:19:04 2008 -0400

    Remove the numVideoScreens xprintism.

    This was to account for cases where you had video and print screens in
    the same server.  Lunacy.  Leave the slot in ScreenInfo, but rename it,
    and stop looking at it.

reqeuestingClient was dropped with the XPRINT drop. Unfortunetely we
are usign it in Image.c. So let's introduce nxagentRequestingClient as
replacement for now.

Backport from xorg-xserver:

  commit 641ce9c706ce3cbf726fb3fc9623161bb96ba127
  Author: George Sapountzis <gsap7@yahoo.gr>
  Date:   Thu Jan 3 17:28:04 2008 +0200

    drop xprint remnants: xpstubs

Needs to be an XlibAtom!