pgindent run? - Mailing list pgsql-hackers

From Robert Haas
Subject pgindent run?
Date
Msg-id CA+TgmoaA9=1RWKtBWpDaj+sF3Stgc8sHgf5z=KGtbjwPLQVDMA@mail.gmail.com
Whole thread Raw
Responses Re: pgindent run?  (Andres Freund <andres@anarazel.de>)
Re: pgindent run?  (Robert Haas <robertmhaas@gmail.com>)
Re: pgindent run?  (Mark Dilger <hornschnorter@gmail.com>)
List pgsql-hackers
If nobody minds too much, I'd like to update typedefs.list and
pgindent the whole tree again.  We've generally done a pretty good job
with pgindenting patches as they are committed this cycle, but we're
starting to accumulate things here and there that are not indented
according to what pgindent likes -- and that makes it harder to keep
later patches indented properly.  You can reindent the files they
touch, but then you have to strip back out the changes that are
unrelated to the patch you're working on.  There are only four source
files where  more than a dozen lines will be touched, so I don't think
this will inconvenience anyone too much:
src/backend/access/transam/xlog.c               | 14 +++---src/backend/tcop/postgres.c                     | 18
+++----src/backend/optimizer/util/relnode.c           | 34 +++++++-------src/backend/catalog/partition.c
| 39 ++++++++--------src/backend/utils/mmgr/generation.c             | 62 ++++++++++++-------------
 

And the changes to typedefs.list would be, as of now, like this:
src/tools/pgindent/typedefs.list                | 53 ++++++++++++++-------

Obviously, one of my motivations here is that I keep ending up
commiting patches to partition.c to fix bugs and add features, and the
fact I've missed that a few times is now coming back to haunt me.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Typo in ExecBuildSlotPartitionKeyDescription prologue
Next
From: Andres Freund
Date:
Subject: Re: pgindent run?