qemu-devel
[Top][All Lists]
Advanced

[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);
> 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]