Re: pgindent run coming

Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > I saw a few odd things.  Most importantly, it seems 'stat' was
> > introduced as a typedef on _both_ lists, yielding weird changes like:
> 
> The standard headers do define "struct stat".  I wonder whether the
> objdump kluge we are using is unable to distinguish typedef names
> from struct tags.
> 
> > I will have to research this further tomorrow.
> 
> We don't have a lot of time for research.  Maybe the best thing is to
> just manually remove stat from the typedef list (along with anything
> else that clearly shouldn't be there)?

The problem is that there are other symbols I don't know about and the
diff is very large. I have found that the problem was caused when we
added Linux support to find_typedef and I have a way to get an accurate
list on my machine.

I will generate a proper list on my machine tomorrow then test Linux
here to see if I can get it to generate the right list too.  But odds
are we are not going to have time to re-run the list on the build farm
even if I can get Linux working here.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +