"warning: declaration of '<something>' shadows a member of 'this'

This shows up in gcc 4.8.5 and has been fixed in gcc 5.0, see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57709

Change the variable names anyway to be on the safe side.

Fixes ArcticaProject/nx-libs#958

In file included from Proxy.h:39:0,
                 from ServerProxy.h:32,
                 from ServerProxy.cpp:36:
Channel.h: In member function 'int Channel::handleEncodeIdentity(EncodeBuffer&, ChannelCache*, MessageStore*, const unsigned char*, unsigned int, int)':
Channel.h:369:3: warning: declaration of 'bigEndian' shadows a member of 'this' [-Wshadow]
   {
   ^
Channel.h: In member function 'int Channel::handleDecodeIdentity(DecodeBuffer&, ChannelCache*, MessageStore*, unsigned char*&, unsigned int&, int, WriteBuffer*)':
Channel.h:378:3: warning: declaration of 'bigEndian' shadows a member of 'this' [-Wshadow]
   {
   ^

RHEL7's g++ 4.8.5 reports this while Debian's g++ 10.2.0-15 does
not. This is described in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57709 and fixed in gcc
5.0.

Rename the variables anyway to be on the safe side.

Fixes ArcticaProject/nx-libs#956

if neither NX_CLIENT ist provided nor /usr/NX/bin/nxclient is existing
try /usr/bin/nxdialog.

Currently the path is hardcoded.

Make nxproxy and nxagent print an arbitrary version number by running
make NX_VERSION_CUSTOM="myvers". nxagent and nxproxy will then show
this in addition to the original version number:

NXPROXY - Version myvers (3.5.99.22)
NXAGENT - Version myvers (3.5.99.22)

Fixes ArcticaProject/nx-libs#606

no more need to pass down a struct, we now only pass the milliseconds
and let the function do the rest.

no more need to pass down a struct, we now only pass the milliseconds
and let the function do the rest.

nxcomp/src/Makefile.am: Correct usage of PTHREAD_CFLAGS and PTHREAD_LIBS. PTHREAD_CFLAGS is also supposed to be used when linking. See comments in m4/ax_pthread.m4.

 This reverts commit 889de46f.

 Fixes ArcticaProject/nx-libs#775.

 Fixes ArcticaProject/nx-libs#777.

The old unix-* session types are deprecated and will always
lead to a warning message "Unrecognized session type".

Internally there are only three session types: agent, proxy
and shadow, all the others are mapped to one of those three.

This fixes the verification in validateSpec()

nxagent: Also provide auto DPI feature via nx/nx option 'autodpi', taking only effect on session startups.

Fixes ArcticaProject/nx-libs#617.

nxcomp/src/Loop.cpp: Make know the agent option 'keyconv' to nxcomp and tolerate it as a valid option.

Fixes ArcticaProject/nx-libs#630.

Fixes ArcticaProject/nx-libs#657.

nxcomp/src/Timestamp.{cpp,h}: use ::ctime_s or ::ctime_r instead of plain ctime, on-stack buffers and return std::string objects.

Fixes: ArcticaProject/nx-libs#616

Fixes ArcticaProject/nx-libs#572

with very few exceptions which require careful thinking ;-)

==28424== 6 bytes in 1 blocks are definitely lost in loss record 1 of 7
==28424==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==28424==    by 0x541D4D9: strndup (strndup.c:43)
==28424==    by 0x4E8AD4B: ChannelEndPoint::getTCPHostAndPort(char**, long*) const (ChannelEndPoint.cpp:311)
==28424==    by 0x4EBE9CC: ConnectToRemote(ChannelEndPoint&) (Loop.cpp:6656)
==28424==    by 0x4EB0A4C: SetupProxyConnection() (Loop.cpp:3205)
==28424==    by 0x4EDC81A: handleNegotiationInLoop(int&, fd_set&, fd_set&, timeval&) (Loop.cpp:14308)
==28424==    by 0x4EAE40F: NXTransPrepare (Loop.cpp:2576)
==28424==    by 0x4EAA801: NXTransContinue (Loop.cpp:1610)
==28424==    by 0x4EB50C9: WaitCleanup() (Loop.cpp:4436)
==28424==    by 0x4EA9507: NXTransProxy (Loop.cpp:1235)
==28424==    by 0x400B2A: main (Main.c:111)

SAFE_FREE has set defaultUnixPath_ to NULL already

==7689== 50 bytes in 5 blocks are definitely lost in loss record 1 of 2
==7689==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==7689==    by 0x54074D9: strndup (strndup.c:43)
==7689==    by 0x4E7D803: ChannelEndPoint::getTCPHostAndPort(char**, long*) const (ChannelEndPoint.cpp:309)
==7689==    by 0x4EC9D93: ConnectToRemote(ChannelEndPoint&) [clone .constprop.144] (Loop.cpp:6660)
==7689==    by 0x4ECB94E: SetupProxyConnection() (Loop.cpp:3204)
==7689==    by 0x4ECE824: handleNegotiationInLoop(int&, fd_set&, fd_set&, timeval&) [clone .isra.129] (Loop.cpp:14312)
==7689==    by 0x4ED0F8A: NXTransPrepare (Loop.cpp:2575)
==7689==    by 0x4ED1C35: NXTransContinue (Loop.cpp:1609)
==7689==    by 0x4ED1D7B: WaitCleanup() (Loop.cpp:4440)
==7689==    by 0x4ED2343: NXTransProxy (Loop.cpp:1234)
==7689==    by 0x400B2A: main (Main.c:111)

Same as in WriteBuffer.cpp

Valgrind will complain about uninitialized data, but we are only
writing to the memory here.

(partially) fixes ArcticaProject/nx-libs#612

(partially) fixes ArcticaProject/nx-libs#612