Re: [PATCH] Add pretty formatting to pg_get_triggerdef

> On Nov 5, 2025, at 04:56, Philip Alger <paalger0@gmail.com> wrote:
>
> Hi Chao,
>
> +/*
> + * pg_get_triggerdef_compact
> + *             Returns trigger definition in a compact, single-line format without
> + *             schema qualification designed for the psql \d command.
> + */
> +Datum
> +pg_get_triggerdef_compact(PG_FUNCTION_ARGS)
> +{
> +       Oid                     trigid = PG_GETARG_OID(0);
> +       char       *res;
> +
> +       res = pg_get_triggerdef_worker(trigid, PRETTYFLAG_SCHEMA);
> ```
>
> I think this is a mis-use of PRETTYFLAG_SCHEMA that is for printing schema-qualified names but omitting schema.
>
> Yes, this is to omit the schema because the functions is used to print out the triggers when using \d in psql, The
currentpractice isn't to print out a schema for the table/view/etc. 
>

I guess I didn’t express myself clearly. My comment was that, this function wants to omit schema, so PRETTYFLAG_SCHEMA
shouldnot be used here, because it is for adding schema. 

>
>
> 3 Looks like you forgot to update pg_get_triggerdef(), when it calls pg_get_triggerdef_worker(tirgid, false), the
secondparameter “false” should be updated to an int flag. 
>
> I fixed this to set to 0. However, false is defined as 0 in stdbool.h
>     #define false 0
>

I know false is actually 0, but passing false to an int argument is misleading.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/