pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
37.10.3
3+ messages / 3 participants
[nested] [flat]

Commits from this thread:
  2020-03-13 dbf95c843a f393fb20a1 7e32aaab58 00f38866f7  Doc: fix mistaken reference to "PG_ARGNULL_xxx()" macro.
* 37.10.3
@ 2020-03-12 06:34 PG Doc comments form <noreply@postgresql.org>
  2020-03-12 23:30 ` Re: 37.10.3 Laurenz Albe <laurenz.albe@cybertec.at>
  0 siblings, 1 reply; 3+ messages in thread

From: PG Doc comments form @ 2020-03-12 06:34 UTC (permalink / raw)
  To: pgsql-docs@lists.postgresql.org; +Cc: mrechte@gmail.com

The following documentation comment has been logged on the website:

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

There is mention to macro PG_ARGNULL_xxx() which I cannot locate. Aren't you
talking of PG_ARGISNULL(n) ?


^ permalink  raw  reply  [nested|flat] 3+ messages in thread

* Re: 37.10.3
  2020-03-12 06:34 37.10.3 PG Doc comments form <noreply@postgresql.org>
@ 2020-03-12 23:30 ` Laurenz Albe <laurenz.albe@cybertec.at>
  2020-03-13 16:22   ` Re: 37.10.3 Tom Lane <tgl@sss.pgh.pa.us>
  0 siblings, 1 reply; 3+ messages in thread

From: Laurenz Albe @ 2020-03-12 23:30 UTC (permalink / raw)
  To: mrechte@gmail.com; pgsql-docs@lists.postgresql.org

On Thu, 2020-03-12 at 06:34 +0000, PG Doc comments form wrote:
> There is mention to macro PG_ARGNULL_xxx() which I cannot locate. Aren't you
> talking of PG_ARGISNULL(n) ?

Indeed.

Here is a patch for that.

Yours,
Laurenz Albe

Attachments:

PG_ARGISNULL.doc.patch (text/x-patch, 729B) download diff --git a/doc/src/sgml/xfunc.sgml b/doc/src/sgml/xfunc.sgml index d9afd3be4d..7c3cfb0115 100644 --- a/doc/src/sgml/xfunc.sgml +++ b/doc/src/sgml/xfunc.sgml @@ -2241,7 +2241,7 @@ PG_FUNCTION_INFO_V1(funcname); <function>PG_GETARG_<replaceable>xxx</replaceable>()</function> macro that corresponds to the argument's data type. In non-strict functions there needs to be a previous check about argument null-ness - using <function>PG_ARGNULL_<replaceable>xxx</replaceable>()</function>. + using <function>PG_ARGISNULL_<replaceable>xxx</replaceable>()</function>. The result is returned using a <function>PG_RETURN_<replaceable>xxx</replaceable>()</function> macro for the return type.
^ permalink raw reply [nested|flat] 3+ messages in thread

* Re: 37.10.3
  2020-03-12 06:34 37.10.3 PG Doc comments form <noreply@postgresql.org>
  2020-03-12 23:30 ` Re: 37.10.3 Laurenz Albe <laurenz.albe@cybertec.at>
@ 2020-03-13 16:22   ` Tom Lane <tgl@sss.pgh.pa.us>
  0 siblings, 0 replies; 3+ messages in thread

From: Tom Lane @ 2020-03-13 16:22 UTC (permalink / raw)
  To: Laurenz Albe <laurenz.albe@cybertec.at>; +Cc: mrechte@gmail.com; pgsql-docs@lists.postgresql.org

Laurenz Albe <laurenz.albe@cybertec.at> writes:
> On Thu, 2020-03-12 at 06:34 +0000, PG Doc comments form wrote:
>> There is mention to macro PG_ARGNULL_xxx() which I cannot locate. Aren't you
>> talking of PG_ARGISNULL(n) ?

> Indeed.

Indeed :-(.

> Here is a patch for that.

Not quite right, it's just PG_ARGISNULL without any type-specific
decoration.  Will push in a moment.

			regards, tom lane





^ permalink  raw  reply  [nested|flat] 3+ messages in thread


end of thread, other threads:[~2020-03-13 16:22 UTC | newest]

Thread overview: 3+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2020-03-12 06:34 37.10.3 PG Doc comments form <noreply@postgresql.org>
2020-03-12 23:30 ` Laurenz Albe <laurenz.albe@cybertec.at>
2020-03-13 16:22   ` Tom Lane <tgl@sss.pgh.pa.us>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox