| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| | |
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
|
|\
| |
| |
| | |
Attributes GH PR #1016: https://github.com/ArcticaProject/nx-libs/pulls
|
|/
|
|
|
|
|
|
|
|
|
|
| |
This should help with clients requesting window manager actions like
maximizing or minimizing. This is a first version as it only handles
messages of type WM_STATE_CHANGE and _NET_WM_STATE. But ICCCM and EWMH
know some more.
The other direction, setting of properties by the WM, is already
implemented in Rootless.c.
Fixes ArcticaProject/nx-libs#1015
|
|\
| |
| |
| |
| |
| | |
into 3.6.x
Attributes GH PR #1017: https://github.com/ArcticaProject/nx-libs/pull/1017
|
| |
| |
| |
| | |
have been moved into bin:pkg xkbcomp-devel.
|
|/
|
|
| |
xorg-x11-xkb-utils-devel have been moved into bin:pkg xkbcomp-devel.
|
|\
| |
| |
| | |
Attributes GH PR #1002: https://github.com/ArcticaProject/nx-libs/pull/1002
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
| |
Increase pylint scrore from 9.5 to 10.0
I: 61, 0: Locally disabling wrong-import-position (C0413) (locally-disabled)
W:119, 0: TODO: Show title item in bold font (fixme)
W:390, 0: TODO: Implement all dialog types (fixme)
C: 41, 0: Trailing whitespace (trailing-whitespace)
C: 50, 0: Line too long (108/100) (line-too-long)
W:158,30: Unused argument '_ypos' (unused-argument)
W:158,23: Unused argument '_xpos' (unused-argument)
W:291,15: Catching too general exception Exception (broad-except)
R:333, 4: Too many branches (15/12) (too-many-branches)
|
|\
| |
| |
| | |
Attributes GH PR #1001: https://github.com/ArcticaProject/nx-libs/pull/1001
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
./nx-X11/x-indent.sh line 5:
INDENT=$(which gnuindent || which gindent || which indent)
^---^ SC2230: which is non-standard. Use builtin 'command -v' instead.
^---^ SC2230: which is non-standard. Use builtin 'command -v' instead.
^---^ SC2230: which is non-standard. Use builtin 'command -v' instead.
For more information:
https://www.shellcheck.net/wiki/SC2230 -- which is non-standard. Use builti...
Signed-off-by: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
$ find . -name "*.sh" | xargs shellcheck -i SC2022
In ./roll-tarballs.sh line 120:
ls -d nx-X11/extras/* | grep -v "nx-X11/extras/tmpMesa*" | xargs rm -r
^----------------------^ SC2022: Note that unlike globs, a* here matches 'aaa' but not 'alpha'.
For more information:
https://www.shellcheck.net/wiki/SC2022 -- Note that unlike globs, a* here m...
Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com>
|
| |
| |
| |
| |
| |
| |
| | |
For more information:
https://www.shellcheck.net/wiki/SC2064 -- Use single quotes, otherwise this...
Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com>
|
|/
|
|
|
|
|
| |
For more information:
https://www.shellcheck.net/wiki/SC2164 -- Use 'cd ... || exit' or 'cd ... |...
Signed-off-by: Mario Trangoni <mjtrangoni@gmail.com>
|