summaryrefslogtreecommitdiffstats
path: root/server/cursor-channel.c
Commit message (Collapse)AuthorAgeFilesLines
* syntax-check: Add missing #include <config.h>Pavel Grunt2015-11-091-0/+4
|
* cursor: fix wrong logic when initializing the channelFabiano Fidêncio2015-11-041-1/+1
| | | | | | | | | | | | | | | | It's a regression introduced by commit e601e920bd5. The logic error was introduced when trying to achieve the following code[0]. but rewritten to prefer an early return, which was wrogly negated. [0]: if (cursor_is_connected(worker) && !COMMON_CHANNEL(worker->cursor_channel)->during_target_migrate) { red_channel_pipes_add_type(RED_CHANNEL(worker->cursor_channel), PIPE_ITEM_TYPE_CURSOR_INIT); } Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Various changes in RedWorker and CursorChannel related to error and warning ↵Marc-André Lureau2015-11-041-8/+20
| | | | | | | messages. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Store QXLInstance in CursorItemMarc-André Lureau2015-11-031-31/+39
| | | | | | | | | | Doing so allows us to remove the extra QXLInstance parameter from cursor_item_unref() and makes the code a bit cleaner. Also add cursor_item_ref(). Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: make more of cursor privateMarc-André Lureau2015-11-031-2/+75
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: make cursor channel privateMarc-André Lureau2015-11-031-1/+23
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* Various minor style changes to worker and cursor channelMarc-André Lureau2015-10-301-15/+20
| | | | | Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* __new_channel -> red_worker_new_channel()Marc-André Lureau2015-10-301-16/+15
| | | | | | | | Rename and lightly refactor the function that creates new common channels for RedWorker (essentially Cursor and Display channels). Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Make cursor_channel_disconnect a CursorChannel methodMarc-André Lureau2015-10-301-1/+3
| | | | | | | | The first argument should be CursorChannel* rather than RedChannel* since it's essentially a CursorChannel method. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Remove a couple single-use static functionsMarc-André Lureau2015-10-301-8/+1
| | | | | | | | | | red_cursor_marshall_inval(), red_migrate_cursor() and on_new_cursor_channel() were short functions that were each only called from a single location, so there's no need for them to be separate functions. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* CursorChannel* arg in cursor_channel_client_new()Marc-André Lureau2015-10-301-3/+2
| | | | | | | | Instead of passing a CommonChannel* argument, use CursorChannel* since this function is only valid for CursorChannels. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* common_channel_client_create -> common_channel_new_clientMarc-André Lureau2015-10-301-9/+9
| | | | | | | | Rename and re-order the initial arguments to make this function look and act more like a method of the CommonChannel class. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Change red_marshall_verb() to accept a VerbItemMarc-André Lureau2015-10-301-2/+2
| | | | | | | Instead of passing a verb enumeration value, pass the verb pipe item Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Remove unused parameter from cursor_channel_new()Marc-André Lureau2015-10-301-1/+1
| | | | | | | | | cursor_channel_new() is only called from one location, and always passes FALSE as the value for the 'migrate' paramater. In addition, this parameter is not used within the function. Remove it. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: move some cursor code to cursor-channel.cMarc-André Lureau2015-10-291-0/+464
Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
generated by cgit (git 2.34.1) at 2024-05-09 13:46:57 +0000