Re: unused code in float8_to_char , formatting.c ? - Mailing list pgsql-hackers

From Robert Haas
Subject Re: unused code in float8_to_char , formatting.c ?
Date
Msg-id CA+TgmoaJTq8cUu6pL2_-8872GnnZ-LLCtH_eWz9yx4yMP=kJQA@mail.gmail.com
Whole thread Raw
In response to Re: unused code in float8_to_char , formatting.c ?  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
On Sat, Nov 30, 2013 at 9:01 PM, Bruce Momjian <bruce@momjian.us> wrote:
> On Sun, Apr  7, 2013 at 12:14:29AM -0400, Robert Haas wrote:
>> On Thu, Apr 4, 2013 at 6:47 PM, Greg Jaskiewicz <gryzman@me.com> wrote:
>> > Looking around the code Today, one of my helpful tools detected this dead code.
>> > As far as I can see, it is actually unused call to strlen() in formatting.c, float8_to_char().
>>
>> I poked at this a little and suggest the following somewhat more
>> extensive cleanup.
>>
>> It seems to me that there are a bunch of these functions where len is
>> unconditionally initialized in NUM_TOCHAR_prepare and then used there.
>>  Similarly in NUM_TOCHAR_cleanup.  And then there's a chunk of each
>> individual function that does it a third time.  Rather than use the
>> same variable in all three places, I've moved the variable
>> declarations to the innermost possible scope.  Doing that revealed a
>> bunch of other, similar places where we can get rid of strlen() calls.
>>
>> Does this version seem like a good idea?
>
> Robert, were you going to apply this patch from April?

Done.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: [PATCH] avoid buffer underflow in errfinish()
Next
From: Robert Haas
Date:
Subject: Re: Logging WAL when updating hintbit