At 2014-07-04 11:34:21 +0200, and...@2ndquadrant.com wrote: > > So we're leaking memory here? For --stats that could well be > relevant...
Will fix (I think using a static buffer would be OK here). > I think we're going to have to redefine the > PGAC_FUNC_SNPRINTF_LONG_LONG_INT_FORMAT callsite in configure.in […] OK, will do. > It's far from guaranteed that 27 will always suffice. I guess it's ok > because it doesn't cause bad breakage, but just some misalignment... Yes, that was my thought too. I could measure the lengths of things and align columns dynamically, but I really doubt it's worth the effort in this case. > Many missing spaces here. […] > Some additional leaking here. Will fix both. > Looks like that should at least partially have been in the other > patch? Yes, sorry. Will fix. (I'll set this back to waiting on author. Thanks for having a look.) -- Abhijit -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers