Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | NX{mitrap,picture}.c: mark NX changes | Ulrich Sibiller | 2020-01-06 | 1 | -1/+4 |
| | |||||
* | Pointer.c: code simplification | Ulrich Sibiller | 2020-01-06 | 1 | -1/+1 |
| | |||||
* | Pointer.c: use __func__ in fprintf | Ulrich Sibiller | 2020-01-06 | 1 | -8/+8 |
| | |||||
* | xkbsrv.h: move extern XkbFreePrivates to central include file | Ulrich Sibiller | 2020-01-06 | 2 | -2/+1 |
| | |||||
* | xkbActions.c: fix formatting | Ulrich Sibiller | 2020-01-06 | 1 | -1/+1 |
| | |||||
* | Window.c: print internal window id as well | Ulrich Sibiller | 2020-01-06 | 1 | -2/+2 |
| | |||||
* | mi: Avoid stack smash when drawing dashed lines | Ulrich Sibiller | 2020-01-06 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | Backport of this xorg-xserver upstream patch: commit 20c2a3bcb11b5baf564e2c73a477ba23f5ae2b10 Author: Peter Harris <pharris@opentext.com> Date: Mon Jul 15 19:44:29 2013 -0400 mi: Avoid stack smash when drawing dashed lines X.org Bug 54013 <https://bugs.freedesktop.org/show_bug.cgi?id=54013> Reviewed-by: Adam Jackson <ajax@redhat.com> Signed-off-by: Peter Harris <pharris@opentext.com> | ||||
* | Clipboard.c: cleanup parentheses | Ulrich Sibiller | 2020-01-05 | 1 | -15/+17 |
| | |||||
* | Clipboard.c: extend/reformat some comments | Ulrich Sibiller | 2020-01-05 | 1 | -8/+12 |
| | |||||
* | Clipboard.c: rename variables/atoms to better reflect their meaning | Ulrich Sibiller | 2020-01-05 | 6 | -40/+43 |
| | | | | | | | Unfortunately we cannot rename NX_CUT_BUFFER_SERVER, too, without breaking compatibility because this one is used to signal nomachine's nxclient after the splash screen is gone (see Splash.c, Window.c and ArticaProject/nx-libs#838) | ||||
* | Clipboard.c: add clientCLIPBOARD variable | Ulrich Sibiller | 2020-01-05 | 1 | -3/+7 |
| | | | | | | the MakeAtom call previously used is as good as using a variable but with the variable it is a) easier to read/understand and b) consistent with the other client* variables. | ||||
* | Clipboard.c: change order of if clause to better match the comment above it | Ulrich Sibiller | 2020-01-05 | 1 | -6/+8 |
| | |||||
* | Clipboard.c: CLINDEX and GetXErrorString are only used for debugging | Ulrich Sibiller | 2020-01-05 | 1 | -2/+2 |
| | |||||
* | Clipboard.c: flush more often to ensure a smooth clipboard experience | Ulrich Sibiller | 2020-01-05 | 1 | -3/+1 |
| | |||||
* | Clipboard.c: fix handling of clipboard=none/client/server/both option | Ulrich Sibiller | 2020-01-05 | 1 | -3/+4 |
| | |||||
* | Options.h: comment on the four clipboard options | Ulrich Sibiller | 2020-01-05 | 1 | -0/+18 |
| | |||||
* | Clipboard.c: Use Find*Index helpers at more locations | Ulrich Sibiller | 2020-01-05 | 1 | -16/+15 |
| | |||||
* | Clipboard.c: cosmetics/typo fix/untabify | Ulrich Sibiller | 2020-01-05 | 2 | -44/+41 |
| | |||||
* | Clipboard.c: introduce helper macro IS_INTERNAL_OWNER | Ulrich Sibiller | 2020-01-05 | 1 | -9/+11 |
| | | | | Using this macro makes the code more readable | ||||
* | Clipboard.c: simplify nxagentSetSelectionOwner | Ulrich Sibiller | 2020-01-05 | 1 | -33/+26 |
| | | | | We do not need to loop over all selections. We have a helper for that. | ||||
* | Clipboard.c: describe nxagentFind*Index | Ulrich Sibiller | 2020-01-05 | 1 | -0/+8 |
| | |||||
* | Clipboard.c: reorder nxagentInitClipboard | Ulrich Sibiller | 2020-01-05 | 1 | -10/+11 |
| | | | | | It makes sense to measure time and set the initialized flag right at the start. | ||||
* | Clipboard.c: clear all selections in nxagentInitClipboard | Ulrich Sibiller | 2020-01-05 | 1 | -2/+4 |
| | | | | | | The new code effectively does the same the old one did. But if we change the number of selections the new code will still work correctly while the old one would not. | ||||
* | Clipboard.c: Fix: re-claim selection on reconnect | Ulrich Sibiller | 2020-01-05 | 1 | -20/+28 |
| | | | | | | | On reconnect claim the selection ownership if one of nxagent's clients is a selection owner. The code for this was already there but could not work because the lastSelectionOwner array was always dropped at the beginning of nxagentInitClipboard. | ||||
* | Clipboard.c: reorder atom allocation code | Ulrich Sibiller | 2020-01-05 | 1 | -10/+10 |
| | | | | | Place the allocation directly before the check for success. Use the same order for server and client atoms. | ||||
* | Clipboard: align reported targets | Ulrich Sibiller | 2020-01-05 | 3 | -13/+26 |
| | | | | | Always report the same list of available targets to internal and external requests. | ||||
* | Clipboard.c: check pointer before usage | Ulrich Sibiller | 2020-01-05 | 1 | -1/+1 |
| | | | | | The code also worked with pWindow being NULL but it did some unnecessary stuff. | ||||
* | Clipboard.c: drop superflous variable | Ulrich Sibiller | 2020-01-05 | 1 | -9/+7 |
| | |||||
* | Clipboard.c: split combined check | Ulrich Sibiller | 2020-01-05 | 1 | -16/+34 |
| | | | | | | By splitting it up we can print appropriate debug messages. More important: The code tried to handle the special targets for all other cases where it should only deny the request. | ||||
* | Clipboard.c: more debugging output | Ulrich Sibiller | 2020-01-05 | 1 | -0/+4 |
| | |||||
* | Clipboard.c: add more explaining comments | Ulrich Sibiller | 2020-01-05 | 1 | -7/+24 |
| | |||||
* | Clipboard.c: use macros instead of hardcoced values for timeouts | Ulrich Sibiller | 2020-01-05 | 1 | -3/+14 |
| | |||||
* | Clipboard.c: fix timeout comparison | Ulrich Sibiller | 2020-01-05 | 1 | -2/+2 |
| | | | | | We had comparisions for >5000 and <5000, but =5000 was not explicitly handled. In that case the code took an unexpected path. | ||||
* | Clipboard: make nxagentInitClipboard return a Boolean | Ulrich Sibiller | 2020-01-05 | 3 | -6/+10 |
| | |||||
* | Clipboard.c: avoid nested extern | Ulrich Sibiller | 2020-01-05 | 1 | -4/+4 |
| | |||||
* | Clipboard.c: fix string formatting directives | Ulrich Sibiller | 2020-01-05 | 1 | -2/+2 |
| | |||||
* | Clipboard.c: Fix formatting | Ulrich Sibiller | 2020-01-05 | 1 | -2/+2 |
| | |||||
* | Clipboard.c, NXproperty.c: match prototypes | Ulrich Sibiller | 2020-01-05 | 2 | -2/+7 |
| | | | | | | The prototype for nxagentGetClipboardWindow has been changed in 63320437ed4e95246ce62193560098f986ac265c but the changes have not made it into NXproperty.c... | ||||
* | Clipboard.c: reformat some comments | Ulrich Sibiller | 2020-01-05 | 1 | -5/+3 |
| | |||||
* | Clipboard.c: add FIXMEs | Ulrich Sibiller | 2020-01-05 | 1 | -0/+3 |
| | |||||
* | Clipboard.c: add nxagentMatchSelectionOwner helper function | Ulrich Sibiller | 2020-01-05 | 1 | -2/+7 |
| | |||||
* | Clipboard.c: make agentClipboardStatus a Boolean | Ulrich Sibiller | 2020-01-05 | 1 | -14/+43 |
| | | | | and add debugging information around its checks. | ||||
* | Clipboard.c: refactor nxagent*SelectionOwner | Ulrich Sibiller | 2020-01-05 | 1 | -28/+28 |
| | | | | | | | expect an index instead of a pointer All calls referenced lastSelectionOwner so let the function directly use it and only pass the required index. | ||||
* | Error.c: simplify some NULL pointer checks | Ulrich Sibiller | 2020-01-05 | 1 | -4/+4 |
| | |||||
* | Error.c: drop now obsolete DEFAULT_STRING_LENGTH | Ulrich Sibiller | 2020-01-05 | 1 | -3/+0 |
| | |||||
* | Error.c: make nxagentRootDir a pointer | Ulrich Sibiller | 2020-01-05 | 1 | -37/+22 |
| | | | | no more hardcoded string length | ||||
* | Error.c: make nxagentHomeDir a pointer | Ulrich Sibiller | 2020-01-05 | 1 | -26/+17 |
| | | | | no more hardcoded string length | ||||
* | Error.c: make nxagentClientsLogName a pointer | Ulrich Sibiller | 2020-01-05 | 3 | -12/+22 |
| | | | | no more hardcoded string length | ||||
* | Error.c: fix format-truncation warning | Ulrich Sibiller | 2020-01-05 | 2 | -40/+19 |
| | | | | | | | | | | Error.c: In function ‘nxagentGetSessionPath’: Error.c:543:62: warning: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 253 [-Wformat-truncation=] snprintf(nxagentSessionDir, DEFAULT_STRING_LENGTH, "%s/C-%s", rootPath, nxagentSessionId); ^~ ~~~~~~~~~~~~~~~~ Error.c:543:5: note: ‘snprintf’ output 4 or more bytes (assuming 259) into a destination of size 256 snprintf(nxagentSessionDir, DEFAULT_STRING_LENGTH, "%s/C-%s", rootPath, nxagentSessionId); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ | ||||
* | Dialog.c: fix format-truncation warning | Ulrich Sibiller | 2020-01-05 | 1 | -4/+14 |
| | | | | | | | | | | Dialog.c: In function ‘nxagentLaunchDialog’: Dialog.c:320:53: warning: ‘%s’ directive output may be truncated writing up to 1023 bytes into a region of size 256 [-Wformat-truncation=] snprintf(dialogDisplay, sizeof(dialogDisplay), "%s", nxagentDisplayName); ^~ ~~~~~~~~~~~~~~~~~~ Dialog.c:320:5: note: ‘snprintf’ output between 1 and 1024 bytes into a destination of size 256 snprintf(dialogDisplay, sizeof(dialogDisplay), "%s", nxagentDisplayName); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |