Make NUM_TOCHAR_prepare and NUM_TOCHAR_finish macros declare "len".
Remove the variable from the enclosing scopes so that nothing can be
relying on it. The net result of this refactoring is that we get rid
of a few unnecessary strlen() calls.
Original patch from Greg Jaskiewicz, substantially expanded by me.
Branch
------
master
Details
-------
http://git.postgresql.org/pg/commitdiff/a8656a3ab02db5b1de0d696dc8cf588efe7c1341
Modified Files
--------------
src/backend/utils/adt/formatting.c | 33 +++++++++++++++------------------
1 file changed, 15 insertions(+), 18 deletions(-)