Re: [PATCH 04/14] reds: Put PipeItem first in VDIReadBuf

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

 



I don't know if something got messed up during a rebase (that doesn't appear to
be the case), but I don't really understand this commit or how the patch relates
to the commit log...

On Thu, 2016-04-07 at 17:11 -0500, Jonathon Jongsma wrote:
> From: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> 
> This is a bit artificial as this is not a real 'PipeItem', but it will
> allow us to make it use the common base class for refcounted pipe items.
> ---
>  server/reds.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 792e45f..1c37627 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -167,9 +167,9 @@ struct ChannelSecurityOptions {
>  };
>  
>  typedef struct VDIReadBuf {
> -    RedCharDeviceVDIPort *dev;
>      RingItem link;
>      uint32_t refs;
> +    RedCharDeviceVDIPort *dev;
>  
>      int len;
>      uint8_t data[SPICE_AGENT_MAX_DATA_SIZE];


Reviewed-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
_______________________________________________
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]