Re: pgsql/ oc/src/sgml/release.sgml rc/backend/com ... - Mailing list pgsql-committers

Tom Lane wrote:
> CVSROOT:    /cvsroot
> Module name:    pgsql
> Changes by:    tgl@postgresql.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