From: | Jon Nelson <jnelson+pgsql(at)jamponi(dot)net> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: minor: contrib/btree_gin/btree_gin.c uses DirectFunctionCall3(inet_in, ..) |
Date: | 2015-03-26 23:02:12 |
Message-ID: | CAKuK5J3vyCvT5h9wcQ5uoR4MR4OCOfOw48a-j_wyM9SiuPtMjw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
On Fri, Mar 20, 2015 at 4:13 PM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> On Fri, Nov 14, 2014 at 01:12:37PM -0600, Jon Nelson wrote:
>> On Fri, Nov 14, 2014 at 11:59 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> > Jon Nelson <jnelson+pgsql(at)jamponi(dot)net> writes:
>> >> contrib/btree_gin/btree_gin.c uses DirectFunctionCall3(inet_in,..)
>> >> instead of DirectFunctionCall1(inet_in, one_argument).
>> >
>> >> That doesn't seem right. Does such a thing matter?
>> >
>> > It's not really incorrect: in a call going through InputFunctionCall(),
>> > which is the normal path, the two extra arguments would be provided
>> > whether the specific datatype input function needed them or not.
>> >
>> > However, I think the usual convention for DirectFunctionCall() usage
>> > is to pass exactly what the target function uses, since you know
>> > exactly what you're calling. Certainly that's what happens in the
>> > two direct calls to inet_in in the core code.
>> >
>> > So I tend to agree that we should change this call to match the others,
>> > but it's purely cosmetic.
>>
>> So, are there any additional steps that you might recommend that I take?
>> It's such a trivial thing. I could provide a patch, of course, or a
>> pull request off of github if people use that.
>
> Patch attached for review.
For as much as it's worth, I did review the patch and it looks just perfect.
--
Jon
From | Date | Subject | |
---|---|---|---|
Next Message | rward | 2015-03-27 04:52:55 | BUG #12908: tstzrange constructor fails when used in WHERE clause |
Previous Message | Kevin Grittner | 2015-03-26 18:47:33 | Re: BUG #12896: Unable to export data from Postgresql database |