typenameTypeId refactoring

Started by Peter Eisentrautover 15 years ago2 messageshackers
Jump to latest
#1Peter Eisentraut
peter_e@gmx.net

Here is a first patch to slightly refactor the type and typmod lookups
with the aim of making it easier to possibly changing the nature or
representation of typmods in the future and making it easier to merge
the collation patch.

I split the typenameTypeId function into two. Most call sites only care
about the type OID; a second function typenameTypeIdAndMod returns type
ID and typmod. So it is more explicit about which places care about
what. It also cuts down on the unholy interface design of returning
half the return value via function result and the other half via an
argument pointer.

Attachments:

typenametypeid.patchtext/x-patch; charset=UTF-8; name=typenametypeid.patchDownload+69-51
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Peter Eisentraut (#1)
Re: typenameTypeId refactoring

Peter Eisentraut <peter_e@gmx.net> writes:

Here is a first patch to slightly refactor the type and typmod lookups
with the aim of making it easier to possibly changing the nature or
representation of typmods in the future and making it easier to merge
the collation patch.

The comment for typenameTypeIdAndMod could use a bit of adjustment
maybe (it's not "only handing back the OID"); otherwise seems fine.

regards, tom lane