Re: [HACKERS] RE: cvs-commit-digest V1 #856

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: "Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp>, "pgsql-hackers" <pgsql-hackers(at)postgreSQL(dot)org>
Subject: Re: [HACKERS] RE: cvs-commit-digest V1 #856
Date: 2000-02-02 03:52:56
Message-ID: 16857.949463576@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)jp> writes:
>> From: Peter Eisentraut - PostgreSQL <petere>
>> Commented out way old printing functions in libpq

> pqbool is removed from libpq-fe.h.
> Couldn't compile interfaces/perl5 now.

> In addition,this seems to change external interface of PQprint().
> Is it OK ?

Not IMHO. It looks like Peter has removed typedef pqbool (potentially
breaking application sources, not just perl5) and changed what were
pqbool == char fields into int fields (thereby breaking binaries that
depend on shared libraries of libpq). Not to mention the advertised
change of removing documented API entry points.

Peter, you need to have a little more respect for stability of
library APIs. Gratuitous breaking of backwards compatibility
is not the done thing around here. It's especially not done
without any discussion.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2000-02-02 04:09:37 Re: [HACKERS] plpgsql problem..
Previous Message Tom Lane 2000-02-02 03:03:06 SELECT FOR UPDATE leaks relation refcounts