Re: pgsql/ oc/src/sgml/release.sgml rc/backend/com ...

Tom Lane wrote:
> CVSROOT: /cvsroot
> Module name: pgsql
> Changes by: tgl(at)postgresql(dot)org 02/07/20 11:12:56
>
> Modified files:
> doc/src/sgml : release.sgml
> src/backend/commands: explain.c
> src/backend/utils/misc: guc.c
> src/include/executor: executor.h
> src/include/utils: guc.h
> src/test/regress/expected: horology-no-DST-before-1970.out
> horology-solaris-1947.out
> horology.out
>
> Log message:
> Code review for SHOW output changes; fix horology expected files for
> new SHOW output format.

Thanks for reviewing and cleaning this up, Tom. I think I understand
most of your changes, but I wasn't sure why you changed
PointerGetDatum(PG_GETARG_TEXT_P(0))
to
PG_GETARG_DATUM(0)

Somewhere I got the impression that the former was preferred, although
the two are equivalent and the latter is more compact.

Joe