| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
As reported by static analyzer:
(error) Common realloc mistake: 'nxagentVisuals' nulled but not freed upon failure
|
|
|
|
|
|
|
| |
As reported by static analyzer:
(error) Common realloc mistake: 'ptr' nulled but not freed upon failure
Attributes ArcticaProject/nx-libs#905
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix these warnings with NX_DEBUG_INPUT enabled:
NXdispatch.c: In function ‘Dispatch’:
NXdispatch.c:350:74: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32’ {aka ‘unsigned int’} [-Wformat=]
fprintf(stderr, "Session: Session started at '%s' timestamp [%lu].\n",
~~^
%u
GetTimeAsString(), GetTimeInMillis());
Reconnect.c: In function ‘nxagentHandleConnectionStates’:
Reconnect.c:303:74: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32’ {aka ‘unsigned int’} [-Wformat=]
fprintf(stderr, "Session: Session suspended at '%s' timestamp [%lu].\n", GetTimeAsString(), GetTimeInMillis());
~~^ ~~~~~~~~~~~~~~~~~
%u
Reconnect.c: In function ‘nxagentReconnectSession’:
Reconnect.c:673:66: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘CARD32’ {aka ‘unsigned int’} [-Wformat=]
fprintf(stderr, "Session: Session resumed at '%s' timestamp [%lu].\n", GetTimeAsString(), GetTimeInMillis());
~~^ ~~~~~~~~~~~~~~~~~
%u
|
|
|
|
|
|
| |
As reported by static analyzer.
Attributes ArcticaProject/nx-libs#905
|
| |
|
| |
|
| |
|
|
|
|
| |
Use the "xorg_" prefix as everywhere else, not "Xorg_".
|
|
|
|
|
| |
Also add original code to make NX changes obvious. Also change some
lines to match original code.
|
|
|
|
|
|
|
| |
and use DEBUG like everywhere else. There were already some TEST lines
using this scheme.
Also indent DEBUG and TEST lines to make the code more readable
|
| |
|
|
|
|
| |
make compilation work with -DNX_DEBUG_INPUT
|
|
|
|
| |
NX_WAKEUP had been non-effective and its effect had been tied to NX_TRANS_WAKEUP.
|
|
|
|
| |
and remove obsolete NX_SPLASH
|
|
|
|
|
|
|
|
| |
Instead of hardcoding the keystrokes in Dialog.h determine the
currently configured keystrokes for the action and insert them into
the dialog strings.
Fixes ArcticaProject/nx-libs#438
|
|
|
|
| |
and use True/False instead of 1/0
|
|
|
|
|
|
| |
This change ensures Literals.h is always referenced during
compilation, effectively hiding it from "unused file" detection we do
occasionally to clean up the source code.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Derived from
commit 978f3b496b9951ee8120a0efcc5cd12503e26770
Author: Roland Mainz <roland.mainz@nrubsig.org>
Date: Sat Mar 5 21:38:29 2005 +0000
xc/programs/Xserver/Imakefile
xc/programs/Xserver/hw/xnest/Imakefile
//bugs.freedesktop.org/show_bug.cgi?id=2653) attachment #2020
(https://bugs.freedesktop.org/attachment.cgi?id=2020): Cleanup Xnest
usage of the DPMS dummy stub functions from dpmsstubs.c instead of
using it's own copy of these functions.
Fixes ArcticaProject/nx-libs#901
|
|
|
|
| |
See ArcticaProject/nx-libs#263 for discussion
|
|
|
|
| |
Instead of 17 XFillPolygon() calls we now use 5.
|
| |
|
|
|
|
| |
Was wrong when scale was different from 1
|
|
|
|
| |
It is not needed elsewhere.
|
| |
|
|
|
|
| |
They are not used from the outside.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
was offset to the right and downwards before
Fixes ArcticaProject/nx-libs#892
|
| |
|
|
|
|
| |
(from nxagent changelog)
|
| |
|
|
|
|
| |
It is better to create the values array once and not on every iteration..
|
|
|
|
|
|
| |
Does not require two parameters. It could be replaced by
RegionDestroy() entirely but we leave it to have a match to
nxagentCreateRegion().
|
|
|
|
|
| |
Now all cleanup code at the end of the function is gone so there's no
need for the gotos anymore. Just return directly.
|
|
|
|
|
| |
move common code into helper function that also takes care of the ugly
alloc/free stuff.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Otherwise we will never be able to free the list because we do not know
if free() is allowed or not.
|
|
|
|
| |
add else clauses, improve debugging, add comments
|
|
|
|
| |
not a real memleak since it is followed by a FatalError...
|
|
|
|
| |
The FIXME just described what the following function actually implements...
|
| |
|
| |
|
| |
|