Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Args.c: do not parse nx/nx marker as option | Ulrich Sibiller | 2020-01-05 | 1 | -9/+32 | |
| | | | | slight optimization | |||||
* | Args.c: fix: do not modify options string | Ulrich Sibiller | 2020-01-05 | 1 | -2/+9 | |
| | | | | | This resulted in a garbled option string on reconnect where everything after the first '=' was lost! | |||||
* | Keyboard.c: scope cleanups | Ulrich Sibiller | 2020-01-05 | 1 | -76/+54 | |
| | ||||||
* | Keyboard.c: Use __func__ at fprintf calls | Ulrich Sibiller | 2020-01-05 | 1 | -51/+50 | |
| | ||||||
* | Keyboard.c: use "\0" instead of "0" | Ulrich Sibiller | 2020-01-05 | 2 | -5/+5 | |
| | ||||||
* | Keyboard.c: fix: handle empty string | Ulrich Sibiller | 2020-01-05 | 1 | -0/+6 | |
| | ||||||
* | Display.c: drop icon file support | Ulrich Sibiller | 2020-01-05 | 2 | -153/+17 | |
| | | | | | | | | | | | | | | | | | | | The normal case was to use the builtin icons. There were two cases where the icon file was read from disk: 1) /usr/NX/share/images/nxagent.xpm was existing which normally is not there 2) case 2 was not true and nxagent.xpm was existing somewhere in the PATH (!) (replace nxagent.xpm by x2goagent.xpm if in x2gp mode) Scanning the path from the PATH variable for xpm files is kind of unexpected and dangerous, too (think of automounter triggers or invalid xpm files). Also remove the xpm files from the distribution. | |||||
* | nxagent.xpm: reduce colours from 242 to 8 | Ulrich Sibiller | 2020-01-05 | 1 | -371/+137 | |
| | | | | | | | | | This leads to a tremendous startup speedup (several seconds, depending on latency) because libXpm had to do a lot of X requests (have not further examined which). With this colour-reduced version things are quick again for those connections without losing a lot of quality. Fixes ArcticaProject/nx-libs#824 | |||||
* | x2goagent.xpm: move copyright to the end to make it a valid xpm file | Ulrich Sibiller | 2020-01-05 | 1 | -16/+15 | |
| | | | | gimp refused to load it, nxagent crashed if this was loaded as icon file! | |||||
* | nxagent manpage: improve descriptions and types | Ulrich Sibiller | 2019-11-24 | 1 | -25/+42 | |
| | | | | | Most <int> options are <bool>. Change that and add missing defaults. Also add list of descriptions for the defer option. | |||||
* | nxagent manpage: extend clipboard documentation | Ulrich Sibiller | 2019-11-24 | 1 | -3/+15 | |
| | ||||||
* | Mesa: silence "maybe uninitialized" warnings | Ulrich Sibiller | 2019-11-18 | 2 | -0/+106 | |
| | ||||||
* | Clipboard.c: report more supported targets | Ulrich Sibiller | 2019-11-15 | 1 | -3/+2 | |
| | | | | | fix one FIXME. This is a rewritten and extended version of a quick patch by Danil Pleshakov and Dimbor. | |||||
* | Clipboard: fix debugging output | Ulrich Sibiller | 2019-11-15 | 1 | -4/+10 | |
| | | | | Correctly determine the Atom names from the real X server. | |||||
* | Clipboard.c: automatically calc the array size | Ulrich Sibiller | 2019-11-15 | 1 | -2/+2 | |
| | ||||||
* | Clipboard.c: improve XChangeProperty code | Ulrich Sibiller | 2019-11-15 | 1 | -2/+2 | |
| | | | | | | | | | | The previously used "Atom" type was confusing. Clarify this by writing the code as requested in the manpage of XChangeProperty: "If the specified format is 8, the property data must be a char array. If the specified format is 16, the property data must be a short array. If the specified format is 32, the property data must be a long array." | |||||
* | Rootless.c: rewrite loop to allow for scope improvements | Ulrich Sibiller | 2019-11-15 | 1 | -17/+10 | |
| | | | | | make i a local loop variable everywhere by changing the while loop to a for loop. | |||||
* | Rootless.c: fix linefeeds when printing lists | Ulrich Sibiller | 2019-11-15 | 1 | -5/+5 | |
| | ||||||
* | Rootless.c: silence compiler warning about wrong parmeter sizes | Ulrich Sibiller | 2019-11-15 | 1 | -5/+3 | |
| | ||||||
* | Rootless.c: disable unused function | Ulrich Sibiller | 2019-11-15 | 1 | -0/+2 | |
| | ||||||
* | Rootless.c: silence compiler warnings about wrong fromat specifiers | Ulrich Sibiller | 2019-11-15 | 1 | -15/+15 | |
| | ||||||
* | Rootless.c: Fatal error if malloc fails | Ulrich Sibiller | 2019-11-15 | 1 | -1/+5 | |
| | ||||||
* | Rootless.c: cleanup function prototypes | Ulrich Sibiller | 2019-11-15 | 1 | -10/+8 | |
| | ||||||
* | Rootless.c: reformat comments | Ulrich Sibiller | 2019-11-15 | 1 | -34/+26 | |
| | ||||||
* | Rootless.c: scope improvements | Ulrich Sibiller | 2019-11-15 | 1 | -58/+38 | |
| | ||||||
* | Rootless.c: use __func__ instead of hardcoded function name | Ulrich Sibiller | 2019-11-15 | 1 | -78/+74 | |
| | ||||||
* | Rootless.c: use designated initializers | Ulrich Sibiller | 2019-11-15 | 1 | -15/+9 | |
| | | | | This fixes a possible unitialized struct variable. | |||||
* | Pixmap.c: reformat comments | Ulrich Sibiller | 2019-11-15 | 1 | -56/+44 | |
| | | | | plus some minor formatting fixes | |||||
* | Pixmap.c: scope improvements | Ulrich Sibiller | 2019-11-15 | 1 | -125/+86 | |
| | ||||||
* | Pixmap.c: drop code that does not make sense | Ulrich Sibiller | 2019-11-15 | 1 | -10/+2 | |
| | | | | | | This check has already been done before we reach here. Also add some comments about a condition that will never occur... | |||||
* | Pixmap.c: call miModifyPixmapHeader twice instead of duplicating code | Ulrich Sibiller | 2019-11-15 | 1 | -83/+11 | |
| | ||||||
* | Args.c: reformat switch blocks | Ulrich Sibiller | 2019-11-15 | 1 | -135/+22 | |
| | | | | This makes them much more readable | |||||
* | Args.c: Unify formatting | Ulrich Sibiller | 2019-11-15 | 1 | -188/+91 | |
| | ||||||
* | Args.c: do not print function name in errors or warnings | Ulrich Sibiller | 2019-11-15 | 1 | -6/+6 | |
| | ||||||
* | Args.c: scope improvements | Ulrich Sibiller | 2019-11-15 | 1 | -29/+16 | |
| | ||||||
* | Args.c: simplify options string allocation | Ulrich Sibiller | 2019-11-15 | 1 | -18/+4 | |
| | ||||||
* | Args.c: fix clipboard=0 or -clipboard 0 | Ulrich Sibiller | 2019-11-15 | 1 | -1/+1 | |
| | | | | This should disable clipboard but effictively did activate clipboard=both. | |||||
* | Args.c: remove unreachable code | Ulrich Sibiller | 2019-11-15 | 1 | -24/+0 | |
| | | | | | | Right at the beginnigng of nxagentParseSingleOption we check for "clipboard" and prepare argv and argc accordingly for ddxProcessArgument. The removed code thus could never be reached. | |||||
* | Args.c: fix unitialized variable | Ulrich Sibiller | 2019-11-15 | 1 | -0/+1 | |
| | | | | | | | "clipboard=something" does not need to be passed on, so return after setting nxagentOptions accordingly. This fixes [nx-X11/programs/Xserver/hw/nxagent/Args.c:1584]: (error) Uninitialized variable: argc | |||||
* | NXglxext.c: use upstream versions of Dispatch functions | Ulrich Sibiller | 2019-11-02 | 2 | -56/+5 | |
| | | | | | | | | | instead of a full copy. We still have our own function because we need to handle the nxagentGlxTrap. This trap is now set before the start of the dispatcher while it has been set only directly before calling the dispatched function. Saves ~50 duplicated lines. | |||||
* | NXwindow.c: use upstream function DeleteWindow() | Ulrich Sibiller | 2019-11-02 | 2 | -52/+0 | |
| | | | | | | | DeleteWindow() is calling FreeWindowResources() which is calling DisposeWindowOptional() which sets pWin->optional to NULL. So the now removed code was never called. After removal DeleteWindow() is identical to the dix version, so we use that one. | |||||
* | NXwindow.c: call InitRootWindow from dix | Ulrich Sibiller | 2019-11-02 | 3 | -49/+18 | |
| | ||||||
* | NXwindow.c: mark NX changes | Ulrich Sibiller | 2019-11-02 | 1 | -0/+16 | |
| | | | | many functions are almost identical to dix. Mark the minimal changes. | |||||
* | NXwindow.c: drop own UnmapWindow() | Ulrich Sibiller | 2019-11-02 | 2 | -18/+0 | |
| | | | | | The only difference to the dix version was a fprintf if compiled with -D TEST. | |||||
* | NXwindow.c: call upstream version UnmapWindow() | Ulrich Sibiller | 2019-11-02 | 2 | -58/+5 | |
| | ||||||
* | NXglyph.c: mark NX changes | Ulrich Sibiller | 2019-11-02 | 1 | -1/+8 | |
| | ||||||
* | NXglyph.c: adapt FindGlyph() to better match render/glyph.c | Ulrich Sibiller | 2019-11-02 | 1 | -19/+24 | |
| | | | | no function change | |||||
* | NXglyphcurs.c: use dixChangeCG() | Ulrich Sibiller | 2019-11-02 | 1 | -18/+13 | |
| | ||||||
* | Simplify (NX)xvdisp.c | Ulrich Sibiller | 2019-11-02 | 3 | -148/+15 | |
| | | | | | | | | | | NXxvdisp.c only exists to set/unset nxagentXvTrap before/after dispatch. There's no need to duplicate the original code. We now rename the original dispatch functions and call them in our dispatch code. Also drop check for sun and cygwin, as they never appeared in xorg upstream code. | |||||
* | NXdispatch.c: mark NX changes | Ulrich Sibiller | 2019-11-02 | 1 | -17/+47 | |
| |