Обсуждение: Re: [PATCHES] include compile problems

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

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. +