This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch to gengtype.c to increase FD soft limit
> From: Phil Edwards <phil@jaj.com>
>
> On Tue, Jun 11, 2002 at 11:08:13AM -0400, Kaveh R. Ghazi wrote:
> > #include "hconfig.h"
> > #include "system.h"
> > +#ifdef HAVE_SYS_RESOURCE_H
> > +# include <sys/resource.h>
> > +#endif
> > #include "gengtype.h"
>
> Loren just discovered a problem with a similar construct in
> libstdc++, where we weren't including enough headers. I don't
> recall what gets included in hconfig.h and system.h, so I can't say
> whether the same problem will happen here, but you might take a
> look at Loren's recent postings to the v3 list to see the problem.
> Phil
I believe system.h DTRT since we're including sys/resource.h in
several other gcc directory files without any problems. Loren's
report (http://gcc.gnu.org/ml/libstdc++/2002-06/msg00092.html if I got
the right one) had to do with unistd.h and sys/time.h being required.
Our system.h takes care of that, but thanks for double checking.
--Kaveh
--
Kaveh R. Ghazi Director of Systems Architecture
ghazi@caip.rutgers.edu Qwest Solutions