From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: Rationalizing code-sharing among src/bin/ directories |
Date: | 2016-03-24 16:11:51 |
Message-ID: | 20160324161151.GA675020@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tom Lane wrote:
> What I propose doing is leaving the above-listed items in
> pg_dump/dumputils.h/.c, and moving the rest of what's in those files
> to new files src/include/fe_utils/string_utils.h and
> src/fe_utils/string_utils.c.
Seems reasonable.
> This name is a bit arbitrary, but most of what's there is string
> processing of some flavor or other, with some list processing thrown
> in for good measure. If anyone's got a different color to paint this
> bikeshed, please speak up.
I wondered about the list stuff while messing about in pg_dump awhile
ago. It seems moderately okay, but not terribly generic; maybe we
should get rid of all that stuff and make ilist.c available to frontend.
Not sure how easy is that, given that AFAIR ilist uses elog. Anyway
maybe we can discuss that in the future, to avoid blocking your patch.
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Yury Zhuravlev | 2016-03-24 16:20:50 | Re: NOT EXIST for PREPARE |
Previous Message | Vladimir Sitnikov | 2016-03-24 16:06:35 | Re: NOT EXIST for PREPARE |