Re: [PATCH spice-server v2] reds-stream: Introduce reds_stream_set_no_delay() helper

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

 



On Mon, Feb 27, 2017 at 12:35:15PM +0000, Frediano Ziglio wrote:
> From: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> 
> The code to enable/disable on a TCP socket is duplicated in multiple
> places in the code base, this commit replaces this duplicated code with
> a helper in RedsStream.
> ---
>  server/common-graphics-channel.c | 15 +++------------
>  server/red-channel-client.c      | 17 ++---------------
>  server/reds-stream.c             | 16 ++++++++++++++++
>  server/reds-stream.h             |  1 +
>  server/reds.c                    |  8 +-------
>  server/sound.c                   |  8 +-------
>  6 files changed, 24 insertions(+), 41 deletions(-)
> 
> Changes since v1:
> - boolean style updates

As far as I'm concerned, NACK to these changes.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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]