| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
This also unifies target handling to never use TEXT or COMPOUND_TEXT
(despite announcing it)
|
|
|
|
| |
This helps in avoiding storing the wrong value in a field.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
there are no negative times...
|
| |
|
| |
|
|
|
|
|
|
| |
if we get an unexpected property print that as an error. Before we saw
"unexpected stage" in that case which does not really help in
debugging...
|
| |
|
|\
| |
| |
| | |
Attributes GH PR #1007: https://github.com/ArcticaProject/nx-libs/pull/1007
|
| | |
|
| |
| |
| |
| | |
They are Booleans for some time now but this change was missing in Trap.h
|
| | |
|
| |
| |
| |
| | |
also correct ugly if formatting regarding nxagentX2go
|
| | |
|
| |
| |
| |
| | |
Use TRUE and FALSE like dix does.
|
| |
| |
| |
| | |
there's no need for that variable to be tri-state
|
| | |
|
| |
| |
| |
| | |
We have it disabled by default but there hasn't been a way to enable it.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
rep->generic.sequenceNumber is of type CARD16
state->sequence is of type unsigned long
Converting state->sequence to an int as it has been done since the
first version of nxcomp I know of (1.3.0-18 from 2003) is wrong here
because for numbers > INT_MAX this will result in a negative number,
which, after applying the 16bit modulo, will not match
rep->generic.sequenceNumber.
Example with numbers:
CARD16 c = 24565
unsigned long u = 3179110389
c % 65536 = 24565
u % 65536 = 24565
(int)(u) = -1115856907
(int)(u) % 65536 = -40971
-40971 will not match 24565
To fix this we need to ensure the number stays positive. We use CARD16
for this to match the type in the request which is a 16bit number. On
my system CARD16 is unsigned short which is guaranteed to contain _at
least_ the 0-65,535 range. As there is no upper limit of the range we
cannot drop the modulo because we need this value to be 16bit and not
more.
Thanks to Norm Green for providing log after log until we could
finally identify the reason for him seeing "Xlib: unexpected async
reply (sequence 0x94b01439)!" when pasting stopped working.
|
| |
| |
| |
| |
| | |
An X reply contains a type which is X_Reply or X_Error. This is not an
opcode which is used when installing the handler.
|
| |
| |
| |
| | |
Should help in debugging "unexpected async reply" problems
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
it was the default for years now, so let's drop the define and include the
code unconditonally.
|
| |
| |
| |
| |
| | |
was never used in the past years, we were always compiling with
-DNXAGENT_SHAPE2
|
| | |
|
| |
| |
| |
| |
| | |
marking all the code that is not really required when not using
nomachine's nxclient.
|
| | |
|
|/ |
|
|\
| |
| |
| | |
Attributes GH PR #1012: https://github.com/ArcticaProject/nx-libs/pull/1012
|
|/
|
|
|
|
| |
The l option of ar in the newer binutils versions switched
from being unused to being used to specify dependencies
so here should be safely removed
|
|\
| |
| |
| | |
Attributes GH PR #1011: https://github.com/ArcticaProject/nx-libs/pull/1011
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Backport of this xorg-xserver commit:
commit 941aeb3b92e644923bd112eef8023f033a140ee6
Author: Olivier Fourdan <ofourdan@redhat.com>
Date: Fri May 13 08:58:58 2016 +0200
randr: Do not update ConnectionInfo if NULL
RRScreenSizeNotify() will update the connection information block, but
if this occurs during initialization before ConnectionInfo is even
initialized, this will lead to a crash.
Simply check for ConnectionInfo prior to update it to avoid the crash.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95337
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
Fixes ArcticaProject/nx-libs#1009
|
|\
| |
| |
| | |
Attributes GH PR #1010: https://github.com/ArcticaProject/nx-libs/pull/1010
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
rep->generic.sequenceNumber is of type CARD16
state->sequence is of type unsigned long
Converting state->sequence to an int as it has been done since the
first version of nxcomp I know of (1.3.0-18 from 2003) is wrong here
because for numbers > INT_MAX this will result in a negative number,
which, after applying the 16bit modulo, will not match
rep->generic.sequenceNumber.
Example with numbers:
CARD16 c = 24565
unsigned long u = 3179110389
c % 65536 = 24565
u % 65536 = 24565
(int)(u) = -1115856907
(int)(u) % 65536 = -40971
-40971 will not match 24565
To fix this we need to ensure the number stays positive. We use CARD16
for this to match the type in the request which is a 16bit number. On
my system CARD16 is unsigned short which is guaranteed to contain _at
least_ the 0-65,535 range. As there is no upper limit of the range we
cannot drop the modulo because we need this value to be 16bit and not
more.
Thanks to Norm Green for providing log after log until we could
finally identify the reason for him seeing "Xlib: unexpected async
reply (sequence 0x94b01439)!" when pasting stopped working.
|
|\
| |
| |
| | |
Attributes GH PR #1008: https://github.com/ArcticaProject/nx-libs/pull/1008
|
| | |
|
|/
|
|
| |
64 bit Cygwin defines __CYGWIN__ instead of __CYGWIN32__
|
|\
| |
| |
| | |
Attributes GH PR #1004: https://github.com/ArcticaProject/nx-libs/pull/1004
|
| |
| |
| |
| | |
It was the only dialog that had a linefeed a the end.
|
| | |
|
| |
| |
| |
| | |
drop some superflous empty lines to make code more compact.
|
| | |
|
| |
| |
| |
| | |
This helps no flooding the output with (double) messages.
|
|/
|
|
|
|
|
|
| |
This ensures the dialog text is visible in the log which is useful
when the dialog display tool (nxclient, nxdialog, ...) fails or
cannot be found.
Fixes ArcticaProject/nx-libs#915
|