Thread: Re: [PATCHES] include compile problems

Re: [PATCHES] include compile problems

From
Tom Lane
Date:
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?

            regards, tom lane

Re: [PATCHES] include compile problems

From
Bruce Momjian
Date:
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. +