35.9.2. Base Types in C-Language Functions

Started by PG Bug reporting formover 5 years ago3 messagesdocs
Jump to latest
#1PG Bug reporting form
noreply@postgresql.org

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/12/xfunc-c.html
Description:

In "Table 35-1. Equivalent C Types for Built-in SQL Types", when SQL type is
"timestamp", the C type is reported to be "Timestamp * ", whereas it is
actually "Timestamp". I experienced it in version 9.3 but I guess it is also
the case in current version.
Best regards

#2Bruce Momjian
bruce@momjian.us
In reply to: PG Bug reporting form (#1)
Re: 35.9.2. Base Types in C-Language Functions

On Wed, Aug 26, 2020 at 01:59:42PM +0000, PG Doc comments form wrote:

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/12/xfunc-c.html
Description:

In "Table 35-1. Equivalent C Types for Built-in SQL Types", when SQL type is
"timestamp", the C type is reported to be "Timestamp * ", whereas it is
actually "Timestamp". I experienced it in version 9.3 but I guess it is also
the case in current version.

Ah, yes, you are right --- Timestamp is an int64. I will fix that with
the attatched patch.

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

The usefulness of a cup is in its emptiness, Bruce Lee

Attachments:

timestamp.difftext/x-diff; charset=us-asciiDownload+1-1
#3Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#2)
Re: 35.9.2. Base Types in C-Language Functions

On Wed, Aug 26, 2020 at 07:09:04PM -0400, Bruce Momjian wrote:

On Wed, Aug 26, 2020 at 01:59:42PM +0000, PG Doc comments form wrote:

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/12/xfunc-c.html
Description:

In "Table 35-1. Equivalent C Types for Built-in SQL Types", when SQL type is
"timestamp", the C type is reported to be "Timestamp * ", whereas it is
actually "Timestamp". I experienced it in version 9.3 but I guess it is also
the case in current version.

Ah, yes, you are right --- Timestamp is an int64. I will fix that with
the attatched patch.

Patch applied.

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

The usefulness of a cup is in its emptiness, Bruce Lee