Re: 'converts internal representation to "..."' comment is confusing - Mailing list pgsql-hackers

From Heikki Linnakangas
Subject Re: 'converts internal representation to "..."' comment is confusing
Date
Msg-id 0aa820db-0536-9074-ea00-4925b12a5d8a@iki.fi
Whole thread Raw
In response to Re: 'converts internal representation to "..."' comment is confusing  (Steve Chavez <steve@supabase.io>)
List pgsql-hackers
On 24/06/2023 23:52, Steve Chavez wrote:
> On Tue, 16 May 2023 at 07:49, Robert Haas <robertmhaas@gmail.com 
> <mailto:robertmhaas@gmail.com>> wrote:
> 
>     On Sun, May 14, 2023 at 9:37 PM Tom Lane <tgl@sss.pgh.pa.us
>     <mailto:tgl@sss.pgh.pa.us>> wrote:
>      > Steve Chavez <steve@supabase.io <mailto:steve@supabase.io>> writes:
>      > > I found "..." confusing in some comments, so this patch changes
>     it to
>      > > "cstring". Which seems to be the intention after all.
>      >
>      > Those comments are Berkeley-era, making them probably a decade older
>      > than the "cstring" pseudotype (invented in b663f3443).  Perhaps what
>      > you suggest is an improvement, but I'm not sure that appealing to
>      > original intent can make the case.
> 
>     FWIW, it does seem like an improvement to me.
> 
> Tom, could we apply this patch since Robert agrees it's an improvement?

Looking around at other input/output functions, we're not very 
consistent, there are many variants of "converts string to [datatype]", 
"converts C string to [datatype]", and "input routine for [datatype]". 
They are all fine, even though they're inconsistent. Doesn't seem worth 
the code churn to change them.

Anyway, I agree this patch is an improvement, so applied. Thanks!

-- 
Heikki Linnakangas
Neon (https://neon.tech)




pgsql-hackers by date:

Previous
From: Tatsuo Ishii
Date:
Subject: Re: Row pattern recognition
Next
From: Andrey Lepikhov
Date:
Subject: Re: Problems with estimating OR conditions, IS NULL on LEFT JOINs