On Wed, Aug 21, 2013 at 3:36 PM, Alex Bligh <[email protected]> wrote:
>
>
> --On 21 August 2013 10:15:52 +0800 Liu Ping Fan <[email protected]> wrote:
>
>> -void slirp_update_timeout(uint32_t *timeout)
>> +static void slirp_update_timeout(uint32_t *timeout)
>> {
>> - if (!QTAILQ_EMPTY(&slirp_instances)) {
>> - *timeout = MIN(1000, *timeout);
>
>
> If you are putting things in macros, you might as well change that
TIMEOUT_FAST/SLOW have definite meaning, and used more than one place
in the code. For 1000ms, I do not know this magic value's meaning, but
whatever, it just occurs once. So there is no trouble to read the
code.
> 1000 as well, and hopefully comment why that particular magic value
> is there.
>
>
>> + Slirp *slirp;
>> + uint32_t t;
>> +
>> + *timeout = MIN(1000, *timeout);
>> + if (*timeout <= TIMEOUT_FAST) {
>> + return;
>> + }
>> + t = *timeout;
>
>
>
>
> --
> Alex Bligh
