Re: [HACKERS] removal of braces

>
> Bruce Momjian <maillist@candle.pha.pa.us> writes:
> > I am sure there were some bugs fixed by this cleanup.
>
> And some introduced, most likely.  Just how much does your script know
> about C syntax, macros, etc?

I have posted the patch to the list.

>
> I concur with David Gould that this is a risky, expensive exercise
> with very little potential gain.
>
> If you were able to identify any spots with the sort of macro-induced
> bug you illustrated, great!  Fix 'em by hand.  But I doubt that a
> 12000-line wholesale modification is worth the trouble.  Heck, half
> the code I've looked at in pgsql doesn't even conform to indentation/
> tab-width conventions ... and *those* we have well-tested tools to fix.

That is more work than fixing all the macros.  I think 98% should be
properly formatted for 4-space tabs.  I run pgindent just before every
beta period, and have added the brace removal code to pgindent.

I got tired of removing them as I saw them.  Now, we have a standard.


--
Bruce Momjian                          |  830 Blythe Avenue
maillist@candle.pha.pa.us              |  Drexel Hill, Pennsylvania 19026
  +  If your life is a hard drive,     |  (610) 353-9879(w)
  +  Christ can be your backup.        |  (610) 853-3000(h)