Add const qualifiers to internal range type APIs

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

This patch adds const qualifiers to internal range type APIs. It
doesn't require any new casts or remove any old ones.

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachments:

0001-Add-const-qualifiers-to-internal-range-type-APIs.patchtext/plain; charset=UTF-8; name=0001-Add-const-qualifiers-to-internal-range-type-APIs.patch; x-mac-creator=0; x-mac-type=0Download+86-87
#2Robert Haas
robertmhaas@gmail.com
In reply to: Peter Eisentraut (#1)
Re: Add const qualifiers to internal range type APIs

On Mon, Oct 28, 2019 at 5:01 AM Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:

This patch adds const qualifiers to internal range type APIs. It
doesn't require any new casts or remove any old ones.

Just out of curiosity, what is the motivation for this?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

#3Peter Eisentraut
peter_e@gmx.net
In reply to: Robert Haas (#2)
Re: Add const qualifiers to internal range type APIs

On 2019-10-28 14:05, Robert Haas wrote:

On Mon, Oct 28, 2019 at 5:01 AM Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:

This patch adds const qualifiers to internal range type APIs. It
doesn't require any new casts or remove any old ones.

Just out of curiosity, what is the motivation for this?

I don't remember. :-)

I had this code lying around from earlier "adventures in const",
probably related to unconstify() and that work, and it seemed sensible
and self-contained enough to finish up and submit.

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

#4Andres Freund
andres@anarazel.de
In reply to: Peter Eisentraut (#3)
Re: Add const qualifiers to internal range type APIs

On 2019-10-29 16:48:24 +0100, Peter Eisentraut wrote:

On 2019-10-28 14:05, Robert Haas wrote:

Just out of curiosity, what is the motivation for this?

I don't remember. :-)

I had this code lying around from earlier "adventures in const", probably
related to unconstify() and that work, and it seemed sensible and
self-contained enough to finish up and submit.

+1

#5Peter Eisentraut
peter_e@gmx.net
In reply to: Andres Freund (#4)
Re: Add const qualifiers to internal range type APIs

On 2019-10-29 21:11, Andres Freund wrote:

On 2019-10-29 16:48:24 +0100, Peter Eisentraut wrote:

On 2019-10-28 14:05, Robert Haas wrote:

Just out of curiosity, what is the motivation for this?

I don't remember. :-)

I had this code lying around from earlier "adventures in const", probably
related to unconstify() and that work, and it seemed sensible and
self-contained enough to finish up and submit.

+1

committed

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services