Re: [PATCHES] include compile problems - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [PATCHES] include compile problems
Date
Msg-id 200607141440.k6EEeVq11747@momjian.us
Whole thread Raw
In response to Re: [PATCHES] include compile problems  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > Patch applied.  Thanks.
>
> I suspect the point was that limits.h is needed *instead of* math.h,
> not *in addition to*.  How many of those headers had math.h before?

The issue was that an include file included another include file that
had math.h, but that include wasn't needed, so when it was removed, a C
file including the first file didn't have math.h anymore.  (I didn't
actually check system include file usage because that is
platform-specific).

Anyway, thanks for the patch you made, and glad you looked at the system
includes, which I did not.

--
  Bruce Momjian   bruce@momjian.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-hackers by date:

Previous
From: Bernd Helmle
Date:
Subject: Re: 8.2 features?
Next
From: "Joshua D. Drake"
Date:
Subject: Re: Fwd: Three weeks left until feature freeze