Greg Stein wrote: > On Fri, Aug 3, 2012 at 3:49 PM, <julianf...@apache.org> wrote: >> ... >> * subversion/libsvn_subr/debug.c >> (debug_pool, debug_file, debug_line): New variables. >> (svn_dbg__preamble): Create a pool, the first time we're called. Just >> store the prefix information instead of printing it. (Note: this >> isn't thread-safe; it already wasn't.) > > Please don't use a pool. Then you get into issues about how large it > may grow. Just snprintf() into a 1k stack-based buffer.
snprintf/vsnprintf isn't in C'89. Use it anyway, or perhaps clear the pool after using it instead? - Julian
