Mercurial > hg > octave-lyh > gnulib-hg
changeset 9461:c6d483077434
Fix inefficiency introduced on 2007-11-03.
author | Bruno Haible <bruno@clisp.org> |
---|---|
date | Fri, 09 Nov 2007 12:52:05 +0100 |
parents | 3d2535323220 |
children | 7bbddc0fae4a |
files | ChangeLog lib/vasnprintf.c |
diffstat | 2 files changed, 14 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2007-11-09 Bruno Haible <bruno@clisp.org> + + * lib/vasnprintf.c (VASNPRINTF): Increase reallocation of snprintf + buffer. Fixes an inefficiency introduced on 2007-11-03. + 2007-11-09 Bruno Haible <bruno@clisp.org> * m4/locale-tr.m4 (gt_LOCALE_TR_UTF8) [BeOS]: Make this test return
--- a/lib/vasnprintf.c +++ b/lib/vasnprintf.c @@ -4294,13 +4294,19 @@ goto overflow; else { - /* Need at least count * sizeof (TCHAR_T) bytes. - But allocate proportionally, to avoid looping + /* Need at least (count + 1) * sizeof (TCHAR_T) + bytes. (The +1 is for the trailing NUL.) + But ask for (count + 2) * sizeof (TCHAR_T) + bytes, so that in the next round, we likely get + maxlen > (unsigned int) count + 1 + and so we don't get here again. + And allocate proportionally, to avoid looping eternally if snprintf() reports a too small count. */ size_t n = xmax (xsum (length, - (count + TCHARS_PER_DCHAR - 1) + ((unsigned int) count + 2 + + TCHARS_PER_DCHAR - 1) / TCHARS_PER_DCHAR), xtimes (allocated, 2));