[Top][All Lists]

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

Re: [PATCH 1/2] inet_ntop: silence gcc warning

From: Paul Eggert
Subject: Re: [PATCH 1/2] inet_ntop: silence gcc warning
Date: Wed, 11 Jan 2012 15:44:16 -0800
User-agent: Mozilla/5.0 (X11; Linux i686; rv:8.0) Gecko/20111124 Thunderbird/8.0

On 01/11/12 15:24, Eric Blake wrote:
> +  best.len = 0;

I have some qualms about adding unnecessary initializations
merely to silence GCC.  It's not just that it bloats the
runtime -- it's that it makes the code more confusing, because
later readers might mistakenly assume that the initializations
are necessary, which might cause them to waste time trying to figure
out why the initializations are there.

How about if we put that assignment inside an "#ifdef lint",
or wrap it in IF_LINT, or something like that?  That should
make it clearer and avoid the runtime bloat.

reply via email to

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