Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > On Friday, November 18, 2011 10:14:22 PM Tom Lane wrote:
> >> Andres Freund <andres(at)anarazel(dot)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.
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
+ It's impossible for everything to be true. +
In response to
pgsql-hackers by date
|Next:||From: Nathan Wagner||Date: 2011-11-28 22:12:20|
|Subject: Re: psql line number reporting from stdin|
|Previous:||From: Alvaro Herrera||Date: 2011-11-28 21:44:05|
|Subject: Re: strange nbtree corruption report|