On 04/10/2011 12:11 PM, Tom Lane wrote:
Robert Haas<robertmh...@gmail.com> writes:
On Sun, Apr 10, 2011 at 11:55 AM, Greg Stark<gsst...@mit.edu> wrote:
Note that in the last one someone carefully made the variable names
line up and pgindent is changing the spacing to an arbitrary amount.
Well, it's the same arbitrary amount that we use throughout our code,
presumably. I am not sure whether pgident is the best tool for the
job, but at least it makes the code relatively consistent throughout,
which is mostly a good thing.
Yes. pgindent has never been about preserving somebody else's idea
of what's appropriate formatting. This is sometimes bad but on the
whole it seems to be a win.
What I was a bit surprised by is the volume of changes in wparser_def.c
--- so far as I can see, that file hardly changed since 9.0, so why did
pgindent suddenly whack it around so much? The other files that changed
a lot are mostly new code so widespread changes are unsurprising.
I had a dim and possibly erroneous recollection that was one of the
files we excluded from pgindent runs. If you look at the history it
hasn't been touched by pgindent since the 8.3 run. But the changes to it
all look fairly kosher at first glance.
cheers
andrew
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
