Re: [PATCH v1 10/10] Remove third argument from red_channel_client_init_send_data()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> This third argument (and the 'item' member of
> RedChannelClient::priv::send_data) was a somewhat roundabout way to keep
> the RedPipeItem alive until a message is sent, just in case some data
> owned by that pipeitem was added to the marshaller by reference. This
> was a rather confusing mechanism, however, since it did not have any
> obvious connection to the _add_by_ref() call. It was never very clear
> whether you needed to pass an item to this function or not. The previous
> series of patches made this parameter unnecessary since the referencing
> of the pipe item (or other related structure) is now more explicitly
> connected to the calls to spice_marshaller_add_by_ref_full().
> ---
>  server/inputs-channel-client.c      |  2 +-
>  server/inputs-channel.c             |  6 +++---
>  server/main-channel-client.c        | 32 ++++++++++++++++----------------
>  server/red-channel-client-private.h |  1 -
>  server/red-channel-client.c         | 17 +----------------
>  server/red-channel-client.h         |  2 +-
>  6 files changed, 22 insertions(+), 38 deletions(-)
> 

It does not compile, there are some missing changes

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]