This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: 3.4 PATCH: Don't use #include "file.h" for system headers inAda


Zack Weinberg writes:

> Rainer Orth <ro@TechFak.Uni-Bielefeld.DE> writes:
> 
> > I noticed that two source files in the gcc/ada subdir use the "" form of
> > #include for system headers.  This is just confusing and shouldn't be
> > necessary.
> 
> ctype.h should not be used at all in GCC, unless this is part of the
> Ada runtime; use safe-ctype.h instead.

AFAIKT, adadecode.c is only linked into gnat1, so the ctype.h can be
removed (since system.h already includes safe-ctype.h) and the single
instance of isdigit converted to ISDIGIT.

This would currently break if IN_GCC isn't defined, but it should always be
(and the !IN_GCC part of the file removed).

I'll submit this patch when it's been tested.

	Rainer


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]