pgindent run?

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: pgindent run?
Date: 2017-11-28 19:51:06
Message-ID: CA+TgmoaA9=1RWKtBWpDaj+sF3Stgc8sHgf5z=KGtbjwPLQVDMA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2017-11-28 19:54:46 Re: pgindent run?
Previous Message Robert Haas 2017-11-28 19:21:06 Re: Typo in ExecBuildSlotPartitionKeyDescription prologue