Re: pgsql: Mention why one C file fails pgindent. - Mailing list pgsql-committers

From Andrew Dunstan
Subject Re: pgsql: Mention why one C file fails pgindent.
Date
Msg-id 4C33A14B.4060103@dunslane.net
Whole thread Raw
In response to pgsql: Mention why one C file fails pgindent.  (momjian@postgresql.org (Bruce Momjian))
Responses Re: pgsql: Mention why one C file fails pgindent.
List pgsql-committers

Bruce Momjian wrote:
> Log Message:
> -----------
> Mention why one C file fails pgindent.
>
> Modified Files:
> --------------
>     pgsql/src/tools/pgindent:
>         README (r1.41 -> r1.42)
>         (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/tools/pgindent/README?r1=1.41&r2=1.42)
>
>

Why not add the file that breaks to
src/tools/pgindent/exclude_file_patterns? (And also maybe the file Tom
just fixed).

And ISTM we should have a list of typdefs for pgindent to ignore anyway.
Currently, the buildfarm ignores:

    'date','interval','timestamp','ANY'

We should probably add 'DECIMAL' to that list.

But really, this list should probably be part of pgindent, not of the
typedef finding code.

cheers

andrew

pgsql-committers by date:

Previous
From: rhaas@postgresql.org (Robert Haas)
Date:
Subject: pgsql: Support setting the keepalive idle time on MacOS X.
Next
From: petere@postgresql.org (Peter Eisentraut)
Date:
Subject: pgsql: Add note that using PL/Python 2 and 3 in the same session will