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]

Re: V3 PATCH: Insert `typename' and `ios_base::' where required


Neil Booth <neil@daikokuya.demon.co.uk> writes:

| Gabriel Dos Reis wrote:-
| 
| > I've just realized that contrary to files include/c_std/**/*.h and
| > include/c_shadow/**/*.h, the files include/bits/*.h don't active the
| > system_header #pragma.  I thought they were.  That is a mistake.
| > 
| > My apologies.
| > 
| > We should activate that pragma, and for maintainers we'll toggle
| > -Wsystem-headers if desirable.
| 
| CPP basically decides system headerness based on the directory the file
| is included in.  Why is that not enough for libstdc++, if things are
| installed correctly?  

For some reasons, that seems not to be sufficient -- maybe because of
V3 include directories hierarchy. 
(for the record, I'm been myself able to reproduce a warning in one of
V3 system header lacking that #pragma).

| ... Or is this a desire to be able to use -I rather
| than -isystem?

Maybe.  What happens when running testsuites with a not-yet-installed
compiler? 

-- Gaby
CodeSourcery, LLC                       http://www.codesourcery.com


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