]> granicus.if.org Git - postgresql/commit
Teach pgindent to skip files generated by bison or flex automatically.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 17 Jun 2017 03:14:27 +0000 (23:14 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 17 Jun 2017 03:14:40 +0000 (23:14 -0400)
commitcea258b63d9c7a6d0a7c5e91e539bb89df4bc078
treed3218290d41cbc5a35d5228e7b649b0153886e8a
parent57fb1d677d98d9c02565e47afdbf5e887b095c9f
Teach pgindent to skip files generated by bison or flex automatically.

If a .c or .h file corresponds to a .y or .l file, skip indenting it.
There's no point in reindenting derived files, and these files tend to
confuse pgindent.  (Which probably indicates a bug in BSD indent, but
I can't get excited about trying to fix it.)

For the same reasons, add src/backend/utils/fmgrtab.c to the set of
files excluded by src/tools/pgindent/exclude_file_patterns.

The point of doing this is that it makes it safe to run pgindent over
the tree without doing "make maintainer-clean" first.  While these are
not the only derived .c/.h files in the tree, they are the only ones
pgindent fails on.  Removing that prerequisite step results in one less
way to mess up a pgindent run, and it's necessary if we ever hope to get
to the ease of running pgindent via "make indent".
src/tools/pgindent/README
src/tools/pgindent/exclude_file_patterns
src/tools/pgindent/pgindent