Re: [PATCH] ecpg: fix progname memory leak

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, Michael Meskes <meskes(at)postgresql(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] ecpg: fix progname memory leak
Date: 2020-10-09 18:57:56
Message-ID: 20201009185756.GB24870@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Oct 9, 2020 at 12:50:36PM -0400, Tom Lane wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
> > That said, having this in a different format would in no way fix the fact
> > that the information is mislabeled and obsolete. It would likely be
> > equally mislabeled and obsolete in an issue tracker. Just look at almost
> > any of the other projects out there that *do* use an issue tracker and have
> > been around for 20+ years.
>
> The long and the short of it is that a list like this is only of value
> if it gets diligently maintained. Putting the data in a different tool
> changes nothing about that. (A tool might reduce the effort involved,
> *if* it's a good match to your workflow. But it won't reduce the effort
> to zero.)
>
> Consulting the wiki page's edit history shows that Bruce has been
> doing some maintenance work, but almost nobody else does. Evidently
> that's not enough.
>
> I'd be +1 for the "aggressive culling" suggested upthread, but
> I'm not particularly volunteering to do it, either.

+1

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EnterpriseDB https://enterprisedb.com

The usefulness of a cup is in its emptiness, Bruce Lee

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2020-10-09 18:59:02 Re: dynamic result sets support in extended query protocol
Previous Message David Christensen 2020-10-09 18:54:01 [PATCH] Add `truncate` option to subscription commands