Tom Lane wrote: > Andres Freund <and...@anarazel.de> writes: > > On Friday, November 18, 2011 10:14:22 PM Tom Lane wrote: > >> Andres Freund <and...@anarazel.de> writes: > >>> Replacing the if chain with if; else if; ... resulted in a small > >>> speedup. Replacing it with a switch() in a bigger one. > > >> Cool, but this patch is impossible to validate by eye. Could you > >> resubmit a version that doesn't reindent unchanged code? Leave it > >> for pgindent to clean that up later. > > > Sure. It was just to confusing reading the code without reindenting. > > Btw, I found git diff/show/blame -w very useful to view reindent code. > > Applied, thanks. Bruce, would you mind running pgindent on > just src/backend/optimizer/util/clauses.c ? That seems safer than > reindenting manually.
Done. -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
