[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/4] Separate inet_connect into inet_connect
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH v4 2/4] Separate inet_connect into inet_connect (blocking) and inet_nonblocking_connect |
Date: |
Mon, 24 Sep 2012 15:37:46 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120827 Thunderbird/15.0 |
On 09/24/2012 12:07 PM, Paolo Bonzini wrote:
> Il 24/09/2012 11:53, Markus Armbruster ha scritto:
>> ongoing
>>
>>>> + * @errp: set in case of an error
>>>> + **/
>> We usually tack function comments to the function definition, not the
>> declaration. Matter of taste. I like it next to the definition,
>> because that improves the comment's chances to get updated along with
>> the function.
>
> Not necessarily, see main-loop.h for example.
>
> /me would like more of these...
too late :)
>
> Paolo
>
>>>> +int inet_connect_opts(QemuOpts *opts, bool block, bool *in_progress,
>>>> + Error **errp);
>
[Qemu-devel] [PATCH v4 3/4] Fix address handling in inet_nonblocking_connect, Orit Wasserman, 2012/09/23
[Qemu-devel] [PATCH v4 4/4] Clear handler only for valid fd, Orit Wasserman, 2012/09/23