Re: bloated heapam.h

From: Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: bloated heapam.h
Date: 2008-05-12 19:52:27
Message-ID: 48289FFB.8010500@sun.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Alvaro Herrera napsal(a):
> Zdenek Kotala wrote:
>> Alvaro Herrera napsal(a):
>>> Zdenek Kotala wrote:
>
>>>> I attached script which should check it. In first step it runs C
>>>> preprocessor on each header (postgres.h is included as well). The
>>>> output from first step is processed again with preprocessor and
>>>> define.h is included.
>
>>> So were you able to detect anything bogus with this technique?
>> No, everything looks OK.
>
> Well, then it is not working, because transam.h is missing from
> bufpage.h ...
>

It tried catch problems with defines not with datatypes. If you mean that
TransactionID is not defined.

I try to play with lint now if it gets better results.

Zdenek

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2008-05-12 20:02:53 Re: bloated heapam.h
Previous Message Tom Lane 2008-05-12 19:47:08 Re: Syntax decisions for pl/pgsql RAISE extension