Thread: pgsql: Clean up libpq's pollution of application namespace by renaming

pgsql: Clean up libpq's pollution of application namespace by renaming

From
tgl@svr1.postgresql.org (Tom Lane)
Date:
Log Message:
-----------
Clean up libpq's pollution of application namespace by renaming the
exported routines of ip.c, md5.c, and fe-auth.c to begin with 'pg_'.
Also get rid of the vestigial fe_setauthsvc/fe_getauthsvc routines
altogether.

Modified Files:
--------------
    pgsql/doc/src/sgml:
        libpq.sgml (r1.193 -> r1.194)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/libpq.sgml.diff?r1=1.193&r2=1.194)
    pgsql/src/backend/commands:
        user.c (r1.161 -> r1.162)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/commands/user.c.diff?r1=1.161&r2=1.162)
    pgsql/src/backend/libpq:
        auth.c (r1.131 -> r1.132)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/auth.c.diff?r1=1.131&r2=1.132)
        crypt.c (r1.66 -> r1.67)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/crypt.c.diff?r1=1.66&r2=1.67)
        hba.c (r1.148 -> r1.149)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/hba.c.diff?r1=1.148&r2=1.149)
        ip.c (r1.31 -> r1.32)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/ip.c.diff?r1=1.31&r2=1.32)
        md5.c (r1.29 -> r1.30)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/md5.c.diff?r1=1.29&r2=1.30)
        pqcomm.c (r1.181 -> r1.182)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/libpq/pqcomm.c.diff?r1=1.181&r2=1.182)
    pgsql/src/backend/postmaster:
        pgstat.c (r1.110 -> r1.111)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/pgstat.c.diff?r1=1.110&r2=1.111)
        postmaster.c (r1.469 -> r1.470)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/postmaster.c.diff?r1=1.469&r2=1.470)
    pgsql/src/backend/utils/adt:
        network.c (r1.55 -> r1.56)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/network.c.diff?r1=1.55&r2=1.56)
        pgstatfuncs.c (r1.25 -> r1.26)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/pgstatfuncs.c.diff?r1=1.25&r2=1.26)
        varlena.c (r1.136 -> r1.137)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/adt/varlena.c.diff?r1=1.136&r2=1.137)
    pgsql/src/include/libpq:
        auth.h (r1.30 -> r1.31)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/libpq/auth.h.diff?r1=1.30&r2=1.31)
        crypt.h (r1.31 -> r1.32)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/libpq/crypt.h.diff?r1=1.31&r2=1.32)
        ip.h (r1.14 -> r1.15)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/libpq/ip.h.diff?r1=1.14&r2=1.15)
    pgsql/src/interfaces/libpq:
        fe-auth.c (r1.105 -> r1.106)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-auth.c.diff?r1=1.105&r2=1.106)
        fe-auth.h (r1.22 -> r1.23)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-auth.h.diff?r1=1.22&r2=1.23)
        fe-connect.c (r1.322 -> r1.323)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/libpq/fe-connect.c.diff?r1=1.322&r2=1.323)

Re: pgsql: Clean up libpq's pollution of application

From
Neil Conway
Date:
On Mon, 2005-17-10 at 13:24 -0300, Tom Lane wrote:
> Clean up libpq's pollution of application namespace by renaming the
> exported routines of ip.c, md5.c, and fe-auth.c to begin with 'pg_'.
> Also get rid of the vestigial fe_setauthsvc/fe_getauthsvc routines
> altogether.

Does this change require that we bump the libpq version number?

-Neil



Neil Conway <neilc@samurai.com> writes:
> On Mon, 2005-17-10 at 13:24 -0300, Tom Lane wrote:
>> Clean up libpq's pollution of application namespace by renaming the
>> exported routines of ip.c, md5.c, and fe-auth.c to begin with 'pg_'.
>> Also get rid of the vestigial fe_setauthsvc/fe_getauthsvc routines
>> altogether.

> Does this change require that we bump the libpq version number?

It shouldn't, because none of these routines are officially part of
the API.  However, perhaps we should anyhow, considering there's also
the PQinitSSL/PQregisterThreadLock business to deal with.

            regards, tom lane

Re: pgsql: Clean up libpq's pollution of application namespace

From
Bruce Momjian
Date:
Tom Lane wrote:
> Neil Conway <neilc@samurai.com> writes:
> > On Mon, 2005-17-10 at 13:24 -0300, Tom Lane wrote:
> >> Clean up libpq's pollution of application namespace by renaming the
> >> exported routines of ip.c, md5.c, and fe-auth.c to begin with 'pg_'.
> >> Also get rid of the vestigial fe_setauthsvc/fe_getauthsvc routines
> >> altogether.
>
> > Does this change require that we bump the libpq version number?
>
> It shouldn't, because none of these routines are officially part of
> the API.  However, perhaps we should anyhow, considering there's also
> the PQinitSSL/PQregisterThreadLock business to deal with.

They are staying, I think. I will add documentation for them.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073