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: Patch: PR 15575: Define HAVE_LANGINFO_CODESET


The macro to define HAVE_LANGINFO_CODESET was removed a while back. GCJ depended on this to determine the default encodings for source code, which is needed to behave like other Java compilers.

Please make sure that this really happens without breaking (when LC_ALL=C) sources that have the odd ISO-8859-1 character in it.


Unfortunately, most likely you need to do some kind of in-band pattern matching -- see the comment in libcpp/charset.c near HAVE_LANGINFO_CODESET, which reports a recent e-mail from Zack to this mailing list.

Since cpplib is always linked in gcc front-ends (due to symtab.c usage in stringpool.c), it may be worth putting the code in cpplib only.

The patch also adds the missing header includes to cppcharset.c so that it compiles with HAVE_LANGINFO_CODESET defined.

Not necessary anymore.


Paolo


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