[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/3] move socket_set_nodelay to osdep.c
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/3] move socket_set_nodelay to osdep.c |
Date: |
Fri, 25 Jan 2013 17:03:51 +0000 |
On Tue, Jan 22, 2013 at 9:36 AM, Stefan Hajnoczi <address@hidden> wrote:
> On Mon, Jan 21, 2013 at 09:23:28AM +0900, MORITA Kazutaka wrote:
>> diff --git a/slirp/tcp_subr.c b/slirp/tcp_subr.c
>> index 1542e43..abc6662 100644
>> --- a/slirp/tcp_subr.c
>> +++ b/slirp/tcp_subr.c
>> @@ -429,8 +429,7 @@ tcp_connect(struct socket *inso)
>> setsockopt(s,SOL_SOCKET,SO_REUSEADDR,(char *)&opt,sizeof(int));
>> opt = 1;
>> setsockopt(s,SOL_SOCKET,SO_OOBINLINE,(char *)&opt,sizeof(int));
>> - opt = 1;
>> - setsockopt(s,IPPROTO_TCP,TCP_NODELAY,(char *)&opt,sizeof(int));
>> + socket_set_nodelay(s);
>
> This function still uses tabs for indentation. The line you added uses
> 4 spaces (QEMU coding style) but now this single line may be
> inconsistent. I suggest using tab for this line so it fits in with the
> existing code.
Don't use tabs but please convert the function to using spaces. There
should be a space after commas too.