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

This commit is the only change between Xorg's libX11 1.6.6 and 1.6.7
that affects our code. So were are effectively now on par with libX11
1.6.7 (commit f3c978476e0be6813268af494efb7ac507451116)

  From: Bhavi Dhingra <b.dhingra@samsung.com>
  Date: Mon, 28 Sep 2015 08:33:40 +0000
  Subject: [PATCH] XcmsLookupColor: fully initialize XColor structs passed to
   _XColor_to_XcmsRGB

  Fixes https://gitlab.freedesktop.org/xorg/lib/libx11/issues/44
  aka https://bugs.freedesktop.org/show_bug.cgi?id=92154Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

We are at X.Org libX11 upstream commit 733f64b

Fixes: ArcticaProject/nx-libs #716, #719 and #720

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

these can be used to determine the PID and name of an X11 client,
e.g. in debugging.

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

One question remains: why are we using "bool" instead of "Bool" here?

has been used as integer sometimes (technically correct, but not nice)

has been used as integer sometimes (technically correct, but not nice)

Fixes ArcticaProject/nx-libs#698

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

Fixes ArcticaProject/nx-libs#301, #631

Backport from xorg-xserver:

  commit ca82d4bddf235c9b68d51d68636bab40eafb9889
  Author: Eric Anholt <eric@anholt.net>
  Date:   Fri Aug 31 13:00:23 2007 -0700

    Bug #7186: Fix an excessive request size limitation that broke big-requests.

    MAXBUFSIZE appears to be a leftover of some previous time.  Instead, just
    use maxBigRequestSize when bigreqs are available (limiting buffers to ~16MB).
    When bigreqs are not available, needed won't be larger than the maximum
    size of a non-bigreqs request (256kB).

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

 Reviewed by Ulrich Sibiller <uli42@gmx.de> -- Fri, 24 Aug 2018 21:47:34 +0000 (UTC)

Call checkX2goAgent much earlier (in ddxProcessArgument) and adapt NXAGENTWINDOWNAMELENGTH to its result.

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

Same as in nxcomp's option handling. We really only need it for ","
(%2C) and "=" (%3D), currently, but it can handle all encoded
characters.

new name is more descriptive

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

We do not need two versions in the code

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

Windowsize was wrong when run with nxagent :<someDisplay>.

Basically three changes:
- reference sizeHints and wmHints correctly (no &)
- do not use uninitialized wmHints
- set wmHints Xutf8SetWMProperties() call instead of separate call

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

Fixes this message when compiling with TEST:

Events.c:2725:22: warning: assignment discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
         message_data = validateString(NameForAtom(x.u.clientMessage.u.l.longs0));

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

"ISO C forbids assignment between function pointer and 'void *' [-pedantic]"

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

 Fixes ArcticaProject/nx-libs#691.

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