Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Merge pull request #1028 from uli42/pr/fix_clipboard_callbacks | Ulrich Sibiller | 2022-01-29 | 3 | -42/+80 | |
|\ | | | | | Pr/fix clipboard callbacks | |||||
| * | NXdispatch.c: use defined value CurrentTime instead of 0 | Ulrich Sibiller | 2021-11-18 | 1 | -2/+4 | |
| | | | | | | | | also add another comment. | |||||
| * | Clipboard.c: fix some comments | Ulrich Sibiller | 2021-11-18 | 1 | -4/+5 | |
| | | ||||||
| * | Events.c: fix typo in comment | Ulrich Sibiller | 2021-11-18 | 1 | -1/+1 | |
| | | ||||||
| * | Clipboard.c: use a TimeStamp datatype for lastTimeChanged as dix does | Ulrich Sibiller | 2021-11-18 | 1 | -15/+6 | |
| | | | | | | | | just to be consistent with dix. | |||||
| * | Clipboard.c: fix typo in array name | Ulrich Sibiller | 2021-11-18 | 1 | -11/+11 | |
| | | ||||||
| * | Clipboard.c: re-add callbacks | Ulrich Sibiller | 2021-11-18 | 1 | -9/+53 | |
| | | | | | | | | | | | | | | | | | | 7aa969cd4ee5fe6ecf74f82442e4a0a7491191c1 dropped calling the callbacks which was a mistake. However, this time we do it without an additional trap. Fixes ArcticaProject/nx-libs#1027 | |||||
* | | Screen.c: fix handling of AllScreens without a running window manager | Ulrich Sibiller | 2021-12-29 | 1 | -0/+1 | |
|/ | | | | | | | | | | | | When used with the old nomachine nxclient (nxclient-3.5.0-9.exe) on Windows for running a desktop session the screen size was wrong (set to 3/4 of the requested size while also activating fullscreen because in that case nomachine's nxwin does not offer a window manager). Fix this by always setting AllScreens to true when there's no window manager, just as it was already done with the FullScreen variable. Fixes: ArcticaProject/nx-libs#1030 | |||||
* | Clipboard.c: fix/reformat some comments | Ulrich Sibiller | 2021-10-30 | 1 | -8/+14 | |
| | ||||||
* | Screen.c, Window.c: use "None" everywhere for FullscreenWindow | Ulrich Sibiller | 2021-10-30 | 2 | -2/+2 | |
| | ||||||
* | Screen.c: Fix wrong format specifier | Ulrich Sibiller | 2021-10-30 | 1 | -1/+1 | |
| | ||||||
* | Imakefiles: whitespace fixes | Ulrich Sibiller | 2021-10-30 | 2 | -37/+36 | |
| | ||||||
* | Args.c: remove superfluous SAFE_free() call | Ulrich Sibiller | 2021-10-30 | 1 | -2/+0 | |
| | ||||||
* | Args.c: fix wrong comment | Ulrich Sibiller | 2021-10-30 | 1 | -3/+2 | |
| | ||||||
* | Args.c: remove some empty lines | Ulrich Sibiller | 2021-10-30 | 1 | -4/+0 | |
| | ||||||
* | nxagent: extend x2go check | Ulrich Sibiller | 2021-10-30 | 3 | -5/+37 | |
| | | | | | | | do not only depend on the program name but also check the DISPLAY variable and the command line parameters. This mainly helps for testing some special constructs when you want to use an alternative nxagent for x2go connections. | |||||
* | Clipboard.c: Add debug output when selection is already cleared | Ulrich Sibiller | 2021-07-05 | 1 | -0/+6 | |
| | ||||||
* | Clipboard.c: check for invalid index | Ulrich Sibiller | 2021-07-05 | 1 | -5/+9 | |
| | | | | | | We only need to check this in nxagentHandleSelectionClearFromXServerByIndex, so we can simplify nxagentHandleSelectionClearFromXServerByAtom. | |||||
* | Events.c: add another comment | Ulrich Sibiller | 2021-07-05 | 1 | -0/+5 | |
| | ||||||
* | Fix Xfixes event handling | Ulrich Sibiller | 2021-07-05 | 5 | -148/+137 | |
| | | | | | | | | | Calling the callback on receptions of an EXTERNAL xfixes event was a weird idea from the start. However, since adding the protection trap around it it made no sense at all because it effectivly made the callback a noop for that case. So let's drop all the trap nonsense and implement it properly. The callback will only be used for actions by internal clients. | |||||
* | NXdispatch.c: add an explaining comment | Ulrich Sibiller | 2021-06-22 | 1 | -3/+8 | |
| | | | | and restructure a bit. | |||||
* | Clipboard.c: print debug message if selection is already cleared | Ulrich Sibiller | 2021-06-21 | 1 | -0/+6 | |
| | ||||||
* | Clipboard.c: Only take full clipboard clearing action if non-empty | Ulrich Sibiller | 2021-06-21 | 1 | -11/+11 | |
| | ||||||
* | Clipboard.c: improve some comments | Ulrich Sibiller | 2021-06-21 | 1 | -3/+10 | |
| | ||||||
* | Clipboard.c: remove superflous check | Ulrich Sibiller | 2021-06-21 | 1 | -14/+6 | |
| | | | | replyPendingRequestSelectionToXServer already checks for None | |||||
* | Clipboard.c: restruct setSelectionOwnerOnXServer | Ulrich Sibiller | 2021-06-21 | 1 | -69/+76 | |
| | | | | Immediately abort if the selection index cannot be determined. | |||||
* | Clipboard.c: do not process requests if we do not have a local owner | Ulrich Sibiller | 2021-06-21 | 1 | -0/+9 | |
| | | | | | | This can happen after having processed a SelectionClear event while the real X server still knows the nxagent serverWindow as the selection owner. | |||||
* | Clipboard.c: extend setSelectionOwnerOnXServer() | Ulrich Sibiller | 2021-06-20 | 1 | -7/+10 | |
| | | | | to let it handle a SelectionClear automatically. | |||||
* | Clipboard.c: remove obsolete (disabled) code | Ulrich Sibiller | 2021-06-20 | 1 | -3/+0 | |
| | | | | | | | Either replyPendingRequest a few lines before will set this to None or it will already be None. Also drop the wrong comment line... | |||||
* | Clipboard.c: add missing target cache invalidation to some (currently ↵ | Ulrich Sibiller | 2021-06-20 | 1 | -0/+2 | |
| | | | | unused) code | |||||
* | Events.c: fix unused variable | Ulrich Sibiller | 2021-06-20 | 1 | -1/+4 | |
| | ||||||
* | Clipboard.c: rename nxagentFindLastSelectionOwnerIndex() | Ulrich Sibiller | 2021-06-20 | 1 | -5/+5 | |
| | | | | Did not refer to LastSelectionOwner at all... | |||||
* | Clipboard.c: rename intermediate variable | Ulrich Sibiller | 2021-06-20 | 1 | -7/+7 | |
| | | | | | Using a capital letter here better reflects that this is an event from the real X server. | |||||
* | Clipboard.c: fix format specifier | Ulrich Sibiller | 2021-06-20 | 1 | -1/+1 | |
| | ||||||
* | Clipboard.c: fix typos in comment | Ulrich Sibiller | 2021-06-20 | 1 | -1/+1 | |
| | ||||||
* | Clipboard.c: Fix: forward the correct type and format | Ulrich Sibiller | 2021-06-20 | 1 | -4/+3 | |
| | | | | now stuff like xclip -o -t LENGTH works (if the owner offers that) | |||||
* | Clipboard.c: improve output for PRINT_CLIPBOARD_CONTENT_ON_DEBUG | Ulrich Sibiller | 2021-06-20 | 1 | -10/+19 | |
| | ||||||
* | Clipboard.c: extend comment | Ulrich Sibiller | 2021-06-20 | 1 | -1/+4 | |
| | ||||||
* | Clipboard.c: add helper variable | Ulrich Sibiller | 2021-06-20 | 1 | -18/+20 | |
| | ||||||
* | Clipboard.c: remove superfluous brackets | Ulrich Sibiller | 2021-06-20 | 1 | -84/+82 | |
| | ||||||
* | Clipboard.c: remove old code | Ulrich Sibiller | 2021-06-20 | 1 | -31/+0 | |
| | | | | | | This block has been unused for a long time now. The comment describing how to reach that path does not make sense anymore because primary and clipboard are not sharing variables anymore. So drop it. | |||||
* | Clipboard.c: harmonize naming regarding internal/local | Ulrich Sibiller | 2021-06-20 | 2 | -162/+162 | |
| | | | | | | Use "local" everywhere instead of a mixture of "local" and "internal". Also replace abbreviations "rem" and "int" by "remote" and "local" because they are confusing readers. | |||||
* | Clipboard.c: use NXAGENT_ONSTART macro on one more location | Ulrich Sibiller | 2021-06-20 | 1 | -0/+2 | |
| | ||||||
* | Clipboard.c: improve some comments as requested by reviewer | Ulrich Sibiller | 2021-06-20 | 1 | -96/+104 | |
| | ||||||
* | NXEvents.c: improve debugging | Ulrich Sibiller | 2021-06-20 | 1 | -0/+3 | |
| | ||||||
* | Clipboard.c: add FIXMEs | Ulrich Sibiller | 2021-06-20 | 1 | -2/+7 | |
| | | | | Also improve code to better match the situation described in the FIXME | |||||
* | Clipboard.c: improve debug output in special case | Ulrich Sibiller | 2021-06-20 | 1 | -1/+4 | |
| | | | | | | This avoids an error message in a special case: nxagentRemoteToLocalAtom: WARNING failed to get name from remote atom. | |||||
* | Clipboard.c: improve comments | Ulrich Sibiller | 2021-06-20 | 1 | -1/+8 | |
| | ||||||
* | Clipboard.c: always flush after XConvertSelection() call | Ulrich Sibiller | 2021-06-20 | 1 | -0/+5 | |
| | | | | This ensures the convert will be sent out immediately. | |||||
* | Clipboard.c: sanity check for invalid resource | Ulrich Sibiller | 2021-06-20 | 1 | -0/+8 | |
| |