Postgres Pro
Downloads
Home   >   mailing lists

Re: [HACKERS] Removing #include "postgres.h" from a couple of headers - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [HACKERS] Removing #include "postgres.h" from a couple of headers
Date
Msg-id CA+TgmoYRE+dPtV7PbCSOZcoGjTpNfGd-dfrHhPbKAJo+Dxv8KQ@mail.gmail.com
Whole thread Raw
In response to [HACKERS] Removing #include "postgres.h" from a couple of headers  (Thomas Munro <thomas.munro@enterprisedb.com>)
Responses Re: [HACKERS] Removing #include "postgres.h" from a couple of headers  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [HACKERS] Removing #include "postgres.h" from a couple of headers  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tree view 
On Wed, Mar 8, 2017 at 3:55 AM, Thomas Munro
<thomas.munro@enterprisedb.com> wrote:
> Over in another thread it was pointed out that a patch I submitted
> broke a project rule by including "postgres.h" in a header.  Here is a
> patch to remove it from dsa.h where I made the same mistake, and also
> a case I found in bufmask.h by grepping.

Committed.

> There are also instances in regcustom.h and snowball's header.h -- are
> those special cases?

I will leave this question to someone wiser (or more self-assured) than I.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: [HACKERS] Skip all-visible pages during second HeapScan of CIC
Next
From: Stephen Frost
Date:
Subject: Re: [HACKERS] Skip all-visible pages during second HeapScan of CIC