pgsql: btree_gin: properly call DirectFunctionCall1()

Started by Bruce Momjianabout 11 years ago5 messageshackers
Jump to latest
#1Bruce Momjian
bruce@momjian.us

btree_gin: properly call DirectFunctionCall1()

Previously we called DirectFunctionCall3() with dummy arguments.

Patch by Jon Nelson

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/843cd0bfe6246d94d9b34a7f36bbb76fdba87b74

Modified Files
--------------
contrib/btree_gin/btree_gin.c | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#1)
Re: pgsql: btree_gin: properly call DirectFunctionCall1()

Bruce Momjian <bruce@momjian.us> writes:

btree_gin: properly call DirectFunctionCall1()
Previously we called DirectFunctionCall3() with dummy arguments.

This patch is entirely wrong and has broken the buildfarm.
Please revert.

regards, tom lane

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#3Bruce Momjian
bruce@momjian.us
In reply to: Tom Lane (#2)
Re: pgsql: btree_gin: properly call DirectFunctionCall1()

On Tue, Mar 24, 2015 at 10:02:03PM -0400, Tom Lane wrote:

Bruce Momjian <bruce@momjian.us> writes:

btree_gin: properly call DirectFunctionCall1()
Previously we called DirectFunctionCall3() with dummy arguments.

This patch is entirely wrong and has broken the buildfarm.
Please revert.

OK, done.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ Everyone has their own god. +

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#4Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#3)
Re: [COMMITTERS] pgsql: btree_gin: properly call DirectFunctionCall1()

On Tue, Mar 24, 2015 at 10:35:10PM -0400, Bruce Momjian wrote:

On Tue, Mar 24, 2015 at 10:02:03PM -0400, Tom Lane wrote:

Bruce Momjian <bruce@momjian.us> writes:

btree_gin: properly call DirectFunctionCall1()
Previously we called DirectFunctionCall3() with dummy arguments.

This patch is entirely wrong and has broken the buildfarm.
Please revert.

OK, done.

OK, I figured out that I was only supposed to change inet_in, not the
other calls to DirectFunctionCall3 (varbit_in and bit_in). Patch
attached.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ Everyone has their own god. +

Attachments:

btree_gin.difftext/x-diff; charset=us-asciiDownload+5-5
#5Tom Lane
tgl@sss.pgh.pa.us
In reply to: Bruce Momjian (#4)
Re: Re: [COMMITTERS] pgsql: btree_gin: properly call DirectFunctionCall1()

Bruce Momjian <bruce@momjian.us> writes:

OK, I figured out that I was only supposed to change inet_in, not the
other calls to DirectFunctionCall3 (varbit_in and bit_in). Patch
attached.

That looks better ...

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers