Bruce Momjian <br...@momjian.us> writes: > On Thu, Jul 18, 2013 at 12:27:21AM -0400, Tom Lane wrote: >> I assert that we should simply remove both of these bits of code, as >> just about every committer on the project is smarter about when to use >> vertical whitespace than this program is.
> OK, I have developed the attached patch that shows the code change of > removing the test for #else/#elif/#endif. You will see that the new > output has odd blank lines for cases like: > #ifndef WIN32 > static int copy_file(const char *fromfile, const char *tofile, bool > force); > --> > #else > static int win32_pghardlink(const char *src, const char *dst); > --> > #endif Hm. So actually, that code is trying to undo excess vertical whitespace that something earlier in pgindent added? Where is that happening? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers