pgsql: Make the to_reg*() functions accept text not cstring. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Make the to_reg*() functions accept text not cstring.
Date
Msg-id E1aGVwY-0002Pu-Uk@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make the to_reg*() functions accept text not cstring.

Using cstring as the input type was a poor decision, because that's not
really a full-fledged type.  In particular, it lacks implicit coercions
from text or varchar, meaning that usages like to_regproc('foo'||'bar')
wouldn't work; basically the only case that did work without explicit
casting was a simple literal constant argument.

The lack of field complaints about this suggests that hardly anyone
is using these functions, so hopefully fixing it won't cause much of
a compatibility problem.  They've only been there since 9.4, anyway.

Petr Korobeinikov

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/ea0d494dae0d3d6fce26bf5d6fbaa07e2ee6c402

Modified Files
--------------
doc/src/sgml/func.sgml           |    2 +-
src/backend/utils/adt/regproc.c  |   16 ++++++++--------
src/include/catalog/catversion.h |    2 +-
src/include/catalog/pg_proc.h    |   16 ++++++++--------
4 files changed, 18 insertions(+), 18 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Make pg_shseclabel available in early backend startup
Next
From: Tom Lane
Date:
Subject: pgsql: In opr_sanity regression test, check for unexpected uses of cstr