Re: include compile problems

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org, Kris Jurka <books(at)ejurka(dot)com>, pgsql-patches(at)postgresql(dot)org
Subject: Re: include compile problems
Date: 2006-07-14 14:40:31
Message-ID: 200607141440.k6EEeVq11747@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)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(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Joshua D. Drake 2006-07-14 14:41:32 Re: Fwd: Three weeks left until feature freeze
Previous Message Bernd Helmle 2006-07-14 14:26:40 Re: 8.2 features?

Browse pgsql-patches by date

  From Date Subject
Next Message Andrew Dunstan 2006-07-14 14:45:16 Re: 8.2 features?
Previous Message Bernd Helmle 2006-07-14 14:26:40 Re: 8.2 features?