Re: pgsql-server/src backend/utils/adt/numutils.c ... - Mailing list pgsql-committers

From Neil Conway
Subject Re: pgsql-server/src backend/utils/adt/numutils.c ...
Date
Msg-id 87ptw40y12.fsf@mailbox.samurai.com
Whole thread Raw
In response to Re: pgsql-server/src backend/utils/adt/numutils.c ...  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: pgsql-server/src backend/utils/adt/numutils.c ...  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-committers
Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Well, not really.  It is more of a tightening up, where '' is not
> really zero, it is ''.

Sure, but "tightening up" is just another way to say "backward
incompatible change". I'd agree this is a change we should make, I'm
just not convinced that it's worth the backward compatibility hit.

What if we just added an elog(WARNING) that said "Zero-length integer
conversion is deprecated and will be removed in the near future", and
then fixed this properly in 7.4? I usually dislike adding warnings,
but I don't think that noting in the documentation that this feature
is deprecated is sufficient in this case: information on this kind of
behavior is nearly impossible to find in the docs.

Cheers,

Neil

--
Neil Conway <neilc@samurai.com> || PGP Key ID: DB3C29FC

pgsql-committers by date:

Previous
From: tgl@postgresql.org (Tom Lane)
Date:
Subject: pgsql-server/src/bin/pg_dump dumputils.h
Next
From: Tom Lane
Date:
Subject: Re: pgsql-server/src backend/utils/adt/numutils.c ...